Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #28062 from vespa-engine/vekterli/post-review-changes | Henning Baldersheim | 2023-08-16 | 7 | -15/+15 |
|\ | | | | | Minor post-review changes | ||||
| * | Take existing map size into account when resizing up | Tor Brede Vekterli | 2023-08-16 | 1 | -1/+1 |
| | | |||||
| * | Use `std::span` with fixed extent instead of `ConstArrayRef` | Tor Brede Vekterli | 2023-08-16 | 2 | -3/+3 |
| | | |||||
| * | Rename destructive statistics read function to be more clear in its semantics | Tor Brede Vekterli | 2023-08-16 | 4 | -11/+11 |
| | | |||||
* | | Go back to vertical listing of long parameter lists. | Henning Baldersheim | 2023-08-16 | 1 | -4/+9 |
| | | |||||
* | | Rename methods to follow style in class | Henning Baldersheim | 2023-08-16 | 5 | -13/+11 |
| | | |||||
* | | GC last use of hasItem, and use lookup interface directly. | Henning Baldersheim | 2023-08-16 | 15 | -58/+50 |
| | | |||||
* | | Check size before checking content. | Henning Baldersheim | 2023-08-16 | 1 | -0/+1 |
| | | |||||
* | | Drop number of anonymous namespaces by grouping related methods together. | Henning Baldersheim | 2023-08-16 | 1 | -46/+34 |
| | | |||||
* | | - Avoid going via a temporary IdealNodesList. | Henning Baldersheim | 2023-08-14 | 17 | -236/+253 |
|/ | | | | | | | - Use ConstArrayRef to hide implementation. - Store all 3 node categories in a single vector. - Use a small_vector that can handle redundancy up to 5 without requiring extra memory allocation. - Build a hash_map if redundancy/groups > 32 for constant lookup time. | ||||
* | When splitting a large number of nodes into their leaf groups you end up ↵ | Henning Baldersheim | 2023-08-11 | 1 | -18/+21 |
| | | | | | | with many vectors with only a single node in them. Then it is more efficient to use a small_vector that keeps the 4 first entries within. Reduces # allocations and avoids indirection. | ||||
* | Merge pull request #28023 from vespa-engine/balder/minor-layout-cleanup | Henning Baldersheim | 2023-08-11 | 3 | -132/+45 |
|\ | | | | | Minor code health. | ||||
| * | Minor code health. | Henning Baldersheim | 2023-08-10 | 3 | -132/+45 |
| | | |||||
* | | Modern c++ for loop. | Henning Baldersheim | 2023-08-10 | 3 | -37/+33 |
| | | |||||
* | | Reduce use of default values in method calls | Henning Baldersheim | 2023-08-10 | 6 | -146/+63 |
| | | |||||
* | | GC unused default value countInvalidAsConsistent | Henning Baldersheim | 2023-08-10 | 4 | -43/+28 |
| | | |||||
* | | Add noexcept | Henning Baldersheim | 2023-08-10 | 3 | -80/+94 |
| | | |||||
* | | Unify code layout | Henning Baldersheim | 2023-08-10 | 1 | -36/+16 |
|/ | |||||
* | Merge pull request #28021 from ↵ | Henning Baldersheim | 2023-08-10 | 12 | -66/+58 |
|\ | | | | | | | | | vespa-engine/balder/generate-fast-idealstate-lookup-once Balder/generate fast idealstate lookup once | ||||
| * | Remove outdated comment | Henning Baldersheim | 2023-08-10 | 1 | -30/+15 |
| | | |||||
| * | Generate fast lookup table for ideal once | Henning Baldersheim | 2023-08-10 | 12 | -36/+43 |
| | | |||||
* | | Merge pull request #28017 from vespa-engine/balder/pass-time-along | Henning Baldersheim | 2023-08-10 | 9 | -92/+76 |
|\ \ | |/ |/| | Balder/pass time along | ||||
| * | Time tracking will not be started until html has been requested at least once. | Henning Baldersheim | 2023-08-10 | 7 | -49/+61 |
| | | |||||
| * | Unify code layout | Henning Baldersheim | 2023-08-10 | 2 | -43/+15 |
| | | |||||
* | | Merge pull request #28011 from vespa-engine/balder/avoid-copying-idealstate | Henning Baldersheim | 2023-08-10 | 6 | -186/+90 |
|\ \ | | | | | | | Balder/avoid copying idealstate | ||||
| * | | No need to check something that will always be false. | Henning Baldersheim | 2023-08-10 | 1 | -1/+1 |
| | | | |||||
| * | | Avoid copying the ideal state out from the ideal state bundle. | Henning Baldersheim | 2023-08-10 | 5 | -185/+89 |
| |/ | | | | | | | It is just used as a const reference. | ||||
* | | Merge pull request #28005 from vespa-engine/balder/use-hash-map | Henning Baldersheim | 2023-08-10 | 5 | -51/+102 |
|\ \ | |/ |/| | - Use hash_map instead of std::unordered_set dur to performance and m… | ||||
| * | Properly reset duration. | Henning Baldersheim | 2023-08-10 | 1 | -1/+1 |
| | | |||||
| * | Clear all members. | Henning Baldersheim | 2023-08-10 | 2 | -1/+3 |
| | | |||||
| * | - Use hash_map instead of std::unordered_set dur to performance and memory ↵ | Henning Baldersheim | 2023-08-09 | 5 | -51/+100 |
| | | | | | | | | | | | | fragmentation. - Move code from .h to .cpp that uses a lot of templated code. | ||||
* | | Merge pull request #28008 from ↵ | Henning Baldersheim | 2023-08-10 | 13 | -68/+57 |
|\ \ | | | | | | | | | | | | | vespa-engine/balder/use-hash_map-for-message-tracking Use faster and lighter hash_map for message tracking in distributor. | ||||
| * | | Move shared_ptrs where possible and reserve space in the vectors once. | Henning Baldersheim | 2023-08-09 | 13 | -59/+47 |
| | | | |||||
| * | | Use faster and lighter hash_map for message tracking in distributor. | Henning Baldersheim | 2023-08-09 | 2 | -9/+10 |
| | | | |||||
* | | | - Since the vespalib::hash_map iterators are invalidated on modification ↵ | Henning Baldersheim | 2023-08-09 | 7 | -20/+16 |
|/ / | | | | | | | | | | | store a unique_ptr in the map and deref the pointer instead. - Also expose vectors as const &. | ||||
* / | GC void ignoreNewFeatures flag always false. | Henning Baldersheim | 2023-08-09 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #27991 from ↵ | Henning Baldersheim | 2023-08-09 | 13 | -20/+20 |
|\ | | | | | | | | | vespa-engine/balder/use-hash-set-for-outdated-nodes Use a vespalib::hash_set<uint8_t> instead of std::unordered_set | ||||
| * | Use a vespalib::hash_set<uint8_t> instead of std::unordered_set | Henning Baldersheim | 2023-08-09 | 13 | -20/+20 |
| | | |||||
* | | Merge pull request #27990 from vespa-engine/balder/minor-noexcept | Henning Baldersheim | 2023-08-09 | 2 | -4/+4 |
|\ \ | | | | | | | Add noexcept and minor style changes | ||||
| * | | Add noexcept and minor style changes | Henning Baldersheim | 2023-08-08 | 2 | -4/+4 |
| | | | |||||
* | | | Merge pull request #27989 from vespa-engine/balder/faster-bucketdb-metrics | Henning Baldersheim | 2023-08-09 | 12 | -68/+59 |
|\ \ \ | | | | | | | | | Move where possible | ||||
| * | | | Use vespalib::hash_map | Henning Baldersheim | 2023-08-08 | 1 | -3/+2 |
| | | | | |||||
| * | | | Unify on a single definition of MinReplicaMap | Henning Baldersheim | 2023-08-08 | 11 | -32/+26 |
| | | | | |||||
| * | | | Move where possible | Henning Baldersheim | 2023-08-08 | 3 | -35/+33 |
| |/ / | |||||
* | | | Merge pull request #27988 from ↵ | Henning Baldersheim | 2023-08-09 | 11 | -69/+34 |
|\ \ \ | |_|/ |/| | | | | | | | | vespa-engine/balder/prepare-for-better-stats-reset-code Balder/prepare for better stats reset code | ||||
| * | | GC unused code | Henning Baldersheim | 2023-08-08 | 3 | -30/+9 |
| | | | |||||
| * | | Use std::array | Henning Baldersheim | 2023-08-08 | 2 | -11/+10 |
| | | | |||||
| * | | Inline simple method | Henning Baldersheim | 2023-08-08 | 2 | -11/+7 |
| | | | |||||
| * | | GC unused code | Henning Baldersheim | 2023-08-08 | 2 | -10/+0 |
| | | | |||||
| * | | Use ConstArrayRef | Henning Baldersheim | 2023-08-08 | 2 | -2/+3 |
| | | |