Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Start the metrics manager after everything has been setup to avoid the ↵ | Henning Baldersheim | 2021-03-19 | 1 | -5/+2 |
| | | | | expensive metrics altered handling. This causes valgrind runs to be way too costly. | ||||
* | Add copyright headers | Jon Bratseth | 2021-03-18 | 1 | -0/+1 |
| | |||||
* | Ensure onStop is proxied. | Henning Baldersheim | 2021-03-17 | 11 | -9/+35 |
| | |||||
* | Only access the meta store in the master thread. | Henning Baldersheim | 2021-03-17 | 1 | -7/+5 |
| | |||||
* | Remove redundant virtual keywords. | Henning Baldersheim | 2021-03-15 | 2 | -10/+10 |
| | |||||
* | Merge pull request #16939 from vespa-engine/toregge/remove-unused-variables | Henning Baldersheim | 2021-03-13 | 1 | -1/+0 |
|\ | | | | | Remove unused variables. | ||||
| * | Remove unused variables. | Tor Egge | 2021-03-13 | 1 | -1/+0 |
| | | |||||
* | | Fix forward declaration. | Tor Egge | 2021-03-13 | 1 | -1/+1 |
|/ | |||||
* | To avoid race condition in test parts of the test code must be run in the ↵ | Henning Baldersheim | 2021-03-10 | 1 | -69/+122 |
| | | | | master thread. | ||||
* | No need to postpone cleanup as we fail fast with the atomic needReschedule flag. | Henning Baldersheim | 2021-03-08 | 2 | -25/+6 |
| | |||||
* | - considerBucket calls cancel already, no need for explicit cancel in ↵ | Henning Baldersheim | 2021-03-07 | 1 | -20/+19 |
| | | | | | | | | failedOperation(). - All inflight buckets are cancelled, and can hence just be cleared. Also removes the need for postponed recompute. - Add sanity check for mover identity before removing it. - Handle failed movers differently. | ||||
* | Erase failed movers, in common place. | Henning Baldersheim | 2021-03-07 | 1 | -4/+4 |
| | |||||
* | - Bring the guards along so they are both created and dropped in the master ↵ | Henning Baldersheim | 2021-03-06 | 4 | -63/+82 |
| | | | | | | thread. - Take the bucketdb guard high up so it is evident where we are holding it. | ||||
* | Revert "Revert "If a document was removed or moved between th emove started ↵ | Henning Baldersheim | 2021-03-05 | 6 | -17/+93 |
| | | | | and the d…"" | ||||
* | Revert "If a document was removed or moved between th emove started and the ↵ | Henning Baldersheim | 2021-03-05 | 6 | -93/+17 |
| | | | | d…" | ||||
* | If a document was removed or moved between th emove started and the document ↵ | Henning Baldersheim | 2021-03-05 | 6 | -17/+93 |
| | | | | | | | | was retrieved there would be a racewith state change or othe bucket changes requiring the bucket to be reconsidered. The mover would appear in sync prior to completing the move in the master thread. This should now be accounted by accounting the failed state in the mover. | ||||
* | Reduce to debug loglevel as sub 16G host will be capped. | Henning Baldersheim | 2021-03-05 | 1 | -2/+2 |
| | |||||
* | all scalars must be double | Håvard Pettersen | 2021-03-04 | 2 | -2/+3 |
| | | | | and all operation results must be at least float | ||||
* | Make the SearchTerm enum a small scoped enum and rename to Type. | Henning Baldersheim | 2021-03-04 | 1 | -1/+1 |
| | |||||
* | Revert "Revert "Set allowedlidbloat to 1 instead of 1000"" | Henning Baldersheim | 2021-03-03 | 1 | -1/+1 |
| | |||||
* | Revert "Set allowedlidbloat to 1 instead of 1000" | Arnstein Ressem | 2021-03-03 | 1 | -1/+1 |
| | |||||
* | Set allowedlidbloat to 1 instead of 1000 | Henning Baldersheim | 2021-03-02 | 1 | -1/+1 |
| | |||||
* | Merge pull request #16739 from ↵ | Henning Baldersheim | 2021-03-02 | 29 | -36/+67 |
|\ | | | | | | | | | vespa-engine/balder/do-not-promise-more-than-you-need Balder/do not promise more than you need | ||||
| * | Reduce visibility and do not provide syncable when not necessary. | Henning Baldersheim | 2021-03-02 | 23 | -21/+52 |
| | | |||||
| * | It is enough to promise a ThreadExecutor, no need to have it syncable. | Henning Baldersheim | 2021-03-02 | 7 | -15/+15 |
| | | |||||
* | | Merge pull request #16741 from ↵ | Henning Baldersheim | 2021-03-02 | 1 | -1/+1 |
|\ \ | |/ |/| | | | | | vespa-engine/vekterli/remove-unused-node-reliability-concept-from-cpp-distribution Remove notion of node-specific reliability from C++ distribution code | ||||
| * | Remove notion of node-specific reliability from C++ distribution code | Tor Brede Vekterli | 2021-03-02 | 1 | -1/+1 |
| | | | | | | | | | | I have never seen this in use anywhere, and can find no code that ever sets it. Bonus is that node candidate trimming can be vastly simplified. | ||||
* | | Sign off that the job is complete by erasing the job token. | Henning Baldersheim | 2021-03-02 | 1 | -4/+5 |
| | | |||||
* | | - Can not move away shared_ptr. | Henning Baldersheim | 2021-03-02 | 1 | -1/+2 |
| | | | | | | | | - Remove the recompute token once complete. | ||||
* | | Postpone buckets for reconsideration until it is safe. | Henning Baldersheim | 2021-03-01 | 2 | -33/+84 |
|/ | |||||
* | Remove unused return value. | Henning Baldersheim | 2021-02-28 | 2 | -4/+3 |
| | |||||
* | Use an enum to avoid dangerous implicit type conversion | Henning Baldersheim | 2021-02-27 | 2 | -2/+2 |
| | |||||
* | Put all params for commit into a single CommitParam struct | Henning Baldersheim | 2021-02-27 | 30 | -140/+133 |
| | |||||
* | Eliminate inline warning. | Tor Egge | 2021-02-27 | 2 | -1/+4 |
| | |||||
* | Fix forward declaration. | Tor Egge | 2021-02-25 | 1 | -1/+1 |
| | |||||
* | Avoid including singlenumericattribute.hpp | Henning Baldersheim | 2021-02-25 | 5 | -6/+0 |
| | |||||
* | ensure we we abort operation if timestamp has changed between start and ↵ | Henning Baldersheim | 2021-02-24 | 3 | -8/+12 |
| | | | | prepare. Also control the lifetime of the keys so they are either destructed or carried over to the next phase (completeMove) to ensure no races for the accounting. | ||||
* | Also verify timestamp | Henning Baldersheim | 2021-02-24 | 1 | -1/+2 |
| | |||||
* | Use current metadata when creating move operation. | Henning Baldersheim | 2021-02-24 | 2 | -15/+36 |
| | |||||
* | Obey stop sign | Henning Baldersheim | 2021-02-23 | 1 | -4/+7 |
| | |||||
* | - Change error handling so that both synchonous and asynchronous errors can ↵ | Henning Baldersheim | 2021-02-23 | 6 | -43/+84 |
| | | | | | | | | be reported back from bucket executor. - Treat remapping as an error. - For lidspace compaction job iterator is reset and will be recreated on next invocation. - For bucketmove th ebucket is rechecked and either discarded or restarted. | ||||
* | Use a value metric for buckets_pending. | Henning Baldersheim | 2021-02-23 | 2 | -2/+2 |
| | |||||
* | Add metirc for buckets pending move. | Henning Baldersheim | 2021-02-23 | 14 | -22/+82 |
| | |||||
* | move getting tensor into the attribute code | Arne Juul | 2021-02-22 | 1 | -20/+3 |
| | | | | | * avoids taking an extra copy of the tensor in the DirectTensorAttribute case | ||||
* | Include memory and typeinfo when needed. | Tor Egge | 2021-02-22 | 1 | -0/+1 |
| | |||||
* | string.h does not need alloc.h | Henning Baldersheim | 2021-02-21 | 14 | -8/+20 |
| | |||||
* | Merge pull request #16608 from ↵ | Henning Baldersheim | 2021-02-21 | 1 | -1/+1 |
|\ | | | | | | | | | vespa-engine/toregge/explicitly-delete-implicitly-deleted-bucket-mover-move-constructor Explicitly delete implicitly deleted BucketMover move constructor. | ||||
| * | Explicitly delete implicitly deleted BucketMover move constructor. | Tor Egge | 2021-02-21 | 1 | -1/+1 |
| | | |||||
* | | Include thread as needed. | Tor Egge | 2021-02-20 | 1 | -0/+1 |
|/ | |||||
* | Only count the operation once. | Henning Baldersheim | 2021-02-19 | 3 | -10/+7 |
| | | | | Update test after code is fixed. |