Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #16241 from vespa-engine/balder/some-code-health | Henning Baldersheim | 2021-01-28 | 14 | -55/+45 |
|\ | | | | | Some code healt by exposing what is necessary only | ||||
| * | Some code healt by exposing what is necessary only | Henning Baldersheim | 2021-01-26 | 14 | -55/+45 |
| | | |||||
* | | Still test both variants | Henning Baldersheim | 2021-01-28 | 1 | -1/+1 |
| | | |||||
* | | In order to ensure that lid stats are sampled at the right time ensure that ↵ | Henning Baldersheim | 2021-01-28 | 9 | -26/+39 |
| | | | | | | | | | | | | | | you do it the *next* time you are in the master thread. This is to ensure that the sync call has taken effect. This also keep the iterator creation logic in one method enhancing readability. | ||||
* | | Ensure all tasks are queued on the master executor so compaction will pack ↵ | Henning Baldersheim | 2021-01-27 | 4 | -2/+9 |
| | | | | | | | | the top too. | ||||
* | | Copy the maintenance subdb so you can safely use its members without ↵ | Henning Baldersheim | 2021-01-27 | 1 | -2/+3 |
| | | | | | | | | worrying that they might change under youor feet. | ||||
* | | Use shared_ptr to ensure lifetime is sufficient. | Henning Baldersheim | 2021-01-27 | 11 | -49/+49 |
| | | |||||
* | | Call onStop on the job | Henning Baldersheim | 2021-01-27 | 1 | -2/+5 |
| | | |||||
* | | Add a proper way for controlled stop. | Henning Baldersheim | 2021-01-27 | 3 | -1/+6 |
| | | |||||
* | | Check _stopped in execute, and guard it by the lock. | Henning Baldersheim | 2021-01-27 | 2 | -7/+13 |
|/ | |||||
* | gc last usages of closures | Henning Baldersheim | 2021-01-26 | 7 | -14/+10 |
| | |||||
* | Kill jobs when syncing feedview. | Henning Baldersheim | 2021-01-26 | 1 | -0/+1 |
| | |||||
* | Remove the syncFeedView upcall as it has no purpose other than forcing it ↵ | Henning Baldersheim | 2021-01-26 | 7 | -34/+3 |
| | | | | | | more than necessary. Instead do it in the main documentdb after all subds are reconfigured. | ||||
* | - Kill maintenance jobs prior to doing reconfig. | Henning Baldersheim | 2021-01-25 | 3 | -17/+25 |
| | | | | - Start if they are stopped. | ||||
* | - Stop maintenance jobs prior to reconfiguring. | Henning Baldersheim | 2021-01-25 | 1 | -14/+9 |
| | | | | - Always create new LidSpaceCompactionHandlers. | ||||
* | Merge pull request #16215 from vespa-engine/balder/use-lambda-over-closure-2 | Henning Baldersheim | 2021-01-25 | 16 | -108/+37 |
|\ | | | | | Closure -> lambda | ||||
| * | Closure -> lambda | Henning Baldersheim | 2021-01-25 | 16 | -108/+37 |
| | | |||||
* | | Use lambda over closure. | Henning Baldersheim | 2021-01-25 | 2 | -19/+18 |
|/ | |||||
* | No need to sync the feedviews when we reconfigure searchable. | Henning Baldersheim | 2021-01-25 | 1 | -2/+2 |
| | |||||
* | Check gid and bucketid inside the correct master thread to ensure you do the ↵ | Henning Baldersheim | 2021-01-25 | 3 | -19/+39 |
| | | | | right thing. | ||||
* | Sync executor when destructing the jobs. | Henning Baldersheim | 2021-01-25 | 2 | -1/+4 |
| | |||||
* | Simplify by avoiding closure. | Henning Baldersheim | 2021-01-25 | 1 | -26/+37 |
| | |||||
* | Do not switch default yet. | Henning Baldersheim | 2021-01-24 | 1 | -1/+1 |
| | |||||
* | - The metastore must be read in the master thread. | Henning Baldersheim | 2021-01-24 | 12 | -47/+34 |
| | | | | - The getMetaData belong to the handler, not the iterator. | ||||
* | Merge pull request #16187 from vespa-engine/balder/use-the-correct-lid | Henning Baldersheim | 2021-01-23 | 2 | -2/+23 |
|\ | | | | | Use the correct lid when looking up the meta data. | ||||
| * | Use the correct lid when looking up the meta data. | Henning Baldersheim | 2021-01-23 | 2 | -2/+23 |
| | | |||||
* | | Use lambda over closure. c++11ification | Henning Baldersheim | 2021-01-23 | 9 | -131/+79 |
|/ | |||||
* | Reverse the logic so it is correct. | Henning Baldersheim | 2021-01-23 | 1 | -7/+7 |
| | |||||
* | Revert "Revert "Wire in the BucketExecutor."" | Henning Baldersheim | 2021-01-23 | 18 | -43/+100 |
| | |||||
* | Revert "Wire in the BucketExecutor." | Harald Musum | 2021-01-23 | 18 | -100/+43 |
| | |||||
* | Wire in config control too. | Henning Baldersheim | 2021-01-23 | 6 | -7/+17 |
| | |||||
* | Update comment and use auto. | Henning Baldersheim | 2021-01-23 | 2 | -4/+4 |
| | |||||
* | Wire in the BucketExecutor. | Henning Baldersheim | 2021-01-23 | 14 | -35/+82 |
| | |||||
* | Merge pull request #16182 from vespa-engine/balder/cleanup-1 | Henning Baldersheim | 2021-01-23 | 10 | -72/+71 |
|\ | | | | | Various readability and code cleanup. closure -> lambda | ||||
| * | Various readability and code cleanup. closure -> lambda | Henning Baldersheim | 2021-01-23 | 10 | -72/+71 |
| | | |||||
* | | Merge pull request #16179 from ↵ | Henning Baldersheim | 2021-01-23 | 2 | -3/+24 |
|\ \ | | | | | | | | | | | | | vespa-engine/balder/make-persistence-engine-bucket-executor-proxy Make the PersistenceEngine a proxy and gatekeeper for the BucketExecu… | ||||
| * | | Make the PersistenceEngine a proxy and gatekeeper for the BucketExecutor ↵ | Henning Baldersheim | 2021-01-23 | 2 | -3/+24 |
| |/ | | | | | | | from the content layer. | ||||
* / | Do not move the lid upon creation, but upon move. | Henning Baldersheim | 2021-01-23 | 1 | -3/+5 |
|/ | |||||
* | Update comments and simplify by using auto | Henning Baldersheim | 2021-01-22 | 4 | -13/+6 |
| | |||||
* | Fix forward declarations. | Tor Egge | 2021-01-22 | 4 | -7/+7 |
| | |||||
* | Merge pull request #16164 from vespa-engine/balder/split-test | Henning Baldersheim | 2021-01-22 | 7 | -550/+821 |
|\ | | | | | - Splitt common test code. | ||||
| * | - Update copyright. | Henning Baldersheim | 2021-01-22 | 6 | -123/+90 |
| | | | | | | | | - Move code to .cpp file. | ||||
| * | - Splitt common test code. | Henning Baldersheim | 2021-01-22 | 7 | -549/+853 |
| | | | | | | | | | | - Splitt out HandlerTest. - Make JobTest parameterized to run with current way of using frozen bucket concept, or future using BucketExecutor and content layer bucket locking. | ||||
* | | Merge pull request #16159 from ↵ | Geir Storli | 2021-01-22 | 6 | -9/+325 |
|\ \ | |/ |/| | | | | | vespa-engine/toregge/propagate-attribute-usage-to-resource-usage-tracker Track attribute resource usage. | ||||
| * | Add get_update_count() method to test class. | Tor Egge | 2021-01-22 | 1 | -4/+5 |
| | | |||||
| * | Remove get_usage_vector() member function from resource usage listener mock. | Tor Egge | 2021-01-22 | 1 | -16/+15 |
| | | | | | | | | Add get_usage() member function to test class. | ||||
| * | Add class comment. | Tor Egge | 2021-01-22 | 1 | -0/+3 |
| | | |||||
| * | Track attribute resource usage. | Tor Egge | 2021-01-22 | 6 | -3/+316 |
| | | |||||
* | | Wire in proper BucketSpace. | Henning Baldersheim | 2021-01-22 | 3 | -8/+15 |
| | | |||||
* | | Make a lidspace job that uses a bucket executor to hold the bucket lock. | Henning Baldersheim | 2021-01-22 | 11 | -195/+399 |
| | |