Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Rename and update comments | Harald Musum | 2022-01-03 | 1 | -21/+20 |
| | |||||
* | Avoid clearing config response queue | Harald Musum | 2022-01-03 | 3 | -14/+21 |
| | | | | | | | There have been races due to the fact that we used to clear the queue when receiving a response, thus missing some of the responses. This stops clearing the queue and handles several items on the queue by polling until the queue is empty when a new item is found on the queue. | ||||
* | Merge pull request #20628 from vespa-engine/hmusum/config-cleanup-3 | gjoranv | 2022-01-03 | 4 | -50/+17 |
|\ | | | | | Simplify MockConnection | ||||
| * | Simplify MockConnection | Harald Musum | 2022-01-03 | 4 | -50/+17 |
| | | | | | | | | | | No need to use thread, start() isn't called anyway. Collapse methods into one to simplify | ||||
* | | Deprecate getConfigMd5() and setConfigMd5() | Harald Musum | 2022-01-03 | 1 | -0/+1 |
|/ | | | | | We don't use md5 checksum for config anymore, methods will be removed in Vespa 8. | ||||
* | GC deprecated junit assertThat. | Henning Baldersheim | 2021-12-21 | 26 | -202/+186 |
| | |||||
* | Fix calculation of initial value of last logged timestamp | Harald Musum | 2021-12-18 | 1 | -2/+2 |
| | |||||
* | Less logging when config requests fail | Harald Musum | 2021-12-16 | 1 | -7/+1 |
| | | | | A warning will be logged every minute independently of this | ||||
* | Merge branch 'master' into ↵ | Harald Musum | 2021-12-11 | 5 | -53/+35 |
|\ | | | | | | | revert-20366-revert-20350-hmusum/config-subscription-refactoring-part-5 | ||||
| * | Merge pull request #20445 from ↵ | Henning Baldersheim | 2021-12-10 | 1 | -1/+1 |
| |\ | | | | | | | | | | | | | vespa-engine/hmusum/wait-longer-before-warning-the-first-time Wait longer before logging a warning the first time | ||||
| | * | Wait longer before logging a warning the first time | Harald Musum | 2021-12-10 | 1 | -1/+1 |
| | | | |||||
| * | | Merge pull request #20431 from vespa-engine/arnej/add-builder-lambda-convenience | Arne H Juul | 2021-12-09 | 2 | -49/+27 |
| |\ \ | | |/ | |/| | Arnej/add builder lambda convenience | ||||
| | * | unit test new functional APIs | Arne H Juul | 2021-12-09 | 2 | -49/+27 |
| | | | |||||
| * | | _executor -> _thread | Henning Baldersheim | 2021-12-09 | 2 | -7/+6 |
| | | | |||||
| * | | Add init_fun to vespalib::Thread too to figure out what the thread is used for. | Henning Baldersheim | 2021-12-09 | 2 | -4/+9 |
| |/ | |||||
* / | Revert "Revert "Config subscription refactoring, part 5 [run-systemtest]"" | Harald Musum | 2021-12-06 | 14 | -167/+136 |
|/ | |||||
* | Revert "Config subscription refactoring, part 5 [run-systemtest]" | Harald Musum | 2021-12-04 | 14 | -136/+167 |
| | |||||
* | Cleanup use of requesters and config sources | Harald Musum | 2021-12-03 | 14 | -149/+136 |
| | | | | | Remove requesters() method from ConfigSubscriber as well, not useful for others and is documented as for internal use only | ||||
* | Remove useless test | Harald Musum | 2021-12-03 | 1 | -18/+0 |
| | | | | We will never use more than one ConfigSourceSet for a GenericConfigSubscriber | ||||
* | Merge pull request #20345 from ↵ | gjoranv | 2021-12-03 | 6 | -24/+26 |
|\ | | | | | | | | | vespa-engine/hmusum/config-subscription-refactoring-part-4 Minor cleanup of ConfigSource in subscriptions [run-systemtest] | ||||
| * | MInor cleanup of ConfigSource in subscriptions | Harald Musum | 2021-12-03 | 6 | -24/+26 |
| | | |||||
* | | track namespace move in documenttypes.def | Arne H Juul | 2021-12-02 | 1 | -1/+1 |
|/ | | | | | | | * For C++ code this introduces a "document::config" namespace, which will sometimes conflict with the global "config" namespace. * Move all forward-declarations of the types DocumenttypesConfig and DocumenttypesConfigBuilder to a common header file. | ||||
* | Merge pull request #20299 from ↵ | Tor Brede Vekterli | 2021-11-30 | 4 | -5/+9 |
|\ | | | | | | | | | vespa-engine/vekterli/preserve-change-flag-for-concurrent-config-updates Preserve change flag for concurrent config updates [run-systemtest] | ||||
| * | Minor cleanups, no changes to semantics | Tor Brede Vekterli | 2021-11-30 | 4 | -5/+5 |
| | | |||||
| * | Preserve changed-flag for concurrent config updates | Tor Brede Vekterli | 2021-11-30 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | This should avoid the following race condition between two config updates arriving within a very short interval: 1. Config generation _n-1_ contains changes to payload 2. Subscriber thread does not apply config _n-1_ internally before generation _n_ arrives 3. Config generation _n_ does not contain changes to payload. Overwrites change flag of generation _n-1_ 4. Change status lost, subscriber does not update internal config | ||||
* | | Remove unnecessary logging | Harald Musum | 2021-11-30 | 1 | -1/+0 |
|/ | |||||
* | Simplify ConfigSubscription | Harald Musum | 2021-11-29 | 8 | -55/+35 |
| | | | | | COnfigSubscriber isn't needed for all subscription (used only in equals method, whoch is only used by tests) | ||||
* | Fix logger name | Harald Musum | 2021-11-29 | 1 | -1/+1 |
| | |||||
* | Add some more logging for file distribution connects | Harald Musum | 2021-11-28 | 1 | -0/+1 |
| | | | | | Sometimes seeing some request fail after a very short time with connection error, not sure what causes it, trying to debug | ||||
* | Make variable volatile | Harald Musum | 2021-11-24 | 1 | -1/+1 |
| | |||||
* | Simplify | Harald Musum | 2021-11-24 | 1 | -8/+5 |
| | |||||
* | Adust when we log warnings a bit to avoid noise | Harald Musum | 2021-11-24 | 1 | -10/+15 |
| | |||||
* | Revive test and test some more | Harald Musum | 2021-11-24 | 2 | -18/+22 |
| | |||||
* | Simplify getting connection to use for file download | Harald Musum | 2021-11-17 | 2 | -3/+3 |
| | |||||
* | Add FileDistributionConnectionPool | Harald Musum | 2021-11-16 | 1 | -2/+2 |
| | | | | | | | FileDistributionConnectionPool is better suited for connections getting files, since each file migh be available on different connections and switching connection for all file downloads if one fails does not make sense, instead one should switch per file download if necessary. | ||||
* | Add some debug logging in ConfigSubscriber | Harald Musum | 2021-11-12 | 1 | -0/+5 |
| | | | | | To be able to see what is going on when getting config in client, how long it takes and to help when rewriting this code | ||||
* | Extend test to cover case where we get new generation, but same config | Harald Musum | 2021-11-11 | 2 | -15/+32 |
| | |||||
* | Implement setNewConfigAndGeneration in GenericJRTConfigSubscription | Harald Musum | 2021-11-10 | 1 | -0/+11 |
| | |||||
* | Revert "Revert "Always update config when generation has changed ↵ | Harald Musum | 2021-11-10 | 2 | -2/+28 |
| | | | | [run-systemtest]"" | ||||
* | Revert "Always update config when generation has changed [run-systemtest]" | Harald Musum | 2021-11-10 | 2 | -28/+2 |
| | |||||
* | Fix | Harald Musum | 2021-11-10 | 1 | -1/+1 |
| | |||||
* | Use Objects.equals for comparison and log if config changed unexpectedly | Harald Musum | 2021-11-10 | 1 | -2/+11 |
| | |||||
* | Always update config when generation has changed | Harald Musum | 2021-11-10 | 2 | -1/+18 |
| | |||||
* | Revert "Always update config when generation has changed [run-systemtest]" | Harald Musum | 2021-11-10 | 1 | -1/+5 |
| | |||||
* | Always update config when generation has changed | Harald Musum | 2021-11-10 | 1 | -5/+1 |
| | |||||
* | Revert "Remove deps to http-utils and add vespajlib where necessary." | gjoranv | 2021-10-21 | 1 | -0/+6 |
| | | | | This reverts commit fe260fe8a98c1d2d19633e16a6308857c9577878. | ||||
* | Reapply "Throw instead of setting config if we get bad config [run-systemtest]" | gjoranv | 2021-10-15 | 1 | -8/+3 |
| | | | | This reverts commit cacd694f88f673a0d6e753c52993df8e46721922. | ||||
* | Remove dead code | Harald Musum | 2021-10-14 | 1 | -7/+0 |
| | |||||
* | Remove unused classes | Harald Musum | 2021-10-14 | 3 | -88/+0 |
| | |||||
* | Remove unused methods | Harald Musum | 2021-10-14 | 6 | -26/+1 |
| |