aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Remove reference to internal servers.Arnstein Ressem2024-02-221-5/+5
|/ / / / /
* | | | | Merge pull request #30375 from ↵Harald Musum2024-02-221-5/+4
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | vespa-engine/hmusum/more-robust-handling-when-creating-dir Create directories in a more robust way
| * | | | Create directories in a more robust wayHarald Musum2024-02-221-5/+4
| | | | |
* | | | | Merge pull request #30381 from ↵Tor Brede Vekterli2024-02-221-7/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/vekterli/less-clever-buffer-management Use temporary byte array when serializing Protobuf in Java
| * | | | | Use temporary byte array when serializing Protobuf in JavaTor Brede Vekterli2024-02-221-7/+4
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous attempt at avoiding unneeded memory allocation and copying was futile because--obviously in retrospect--the underlying byte buffer is fixed in size and not dynamically growable. This did not manifest itself in any unit tests (too little data) or the set of system tests that I ran manually. It seems likely that we want to reconsider the encode/decode APIs in the `DocumentProtocol` to allow for more optimal memory management.
* | | | | Merge pull request #30380 from ↵Tor Brede Vekterli2024-02-2211-1359/+867
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/vekterli/move-docapi-message-tests-to-gtest Move C++ MessageBus DocumentAPI message-related tests to GTest
| * | | | | Include file names in IO failure exceptionsTor Brede Vekterli2024-02-221-2/+2
| | | | | |
| * | | | | Move C++ DocumentAPI message tests to GTestTor Brede Vekterli2024-02-2211-1359/+867
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Message-specific test cases are no longer delegated to a quasi-framework in a parent class, but implemented with regular test case functions. Clean up and move existing `TestBase` into a dedicated `MessageFixture` class. Use `std::filesystem::path` instead of plain strings for file paths. This also merges 3 standalone test apps into 1 GTest runner.
* | | | | | Merge pull request #30359 from ↵Jon Bratseth2024-02-2218-74/+149
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/arnej/allow-declaring-string-type-input allow inputs { query(foo) string }
| * | | | | | allow "long" with warningArne Juul2024-02-221-0/+7
| | | | | | |
| * | | | | | demand newline between declarations of inputsArne Juul2024-02-226-20/+15
| | | | | | |
| * | | | | | drop debug printArne Juul2024-02-221-7/+0
| | | | | | |
| * | | | | | allow inputs { query(foo) string }Arne Juul2024-02-2213-54/+134
| | |_|/ / / | |/| | | |
* | | | | | Merge pull request #30358 from ↵Geir Storli2024-02-221-56/+91
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/geirst/filter-simulation-in-iterator-benchmark Support simulation of a filter in non-strict context.
| * | | | | | Support simulation of a filter in non-strict context.Geir Storli2024-02-211-56/+91
| | |_|_|_|/ | |/| | | |
* | | | | | Merge pull request #30379 from ↵Geir Storli2024-02-225-361/+68
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | vespa-engine/toregge/remove-btree-store-insert-and-remove Remove BTreeStore insert and remove member functions, use apply instead.
| * | | | | Remove BTreeStore insert and remove member functions, use apply instead.Tor Egge2024-02-225-361/+68
|/ / / / /
* | | | | Merge pull request #30367 from vespa-engine/renovate/org.jetbrains.intellij-1.xHenning Baldersheim2024-02-221-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Update plugin org.jetbrains.intellij to v1.17.2
| * | | | | Update plugin org.jetbrains.intellij to v1.17.2renovate[bot]2024-02-211-1/+1
| | | | | |
* | | | | | Merge pull request #30369 from vespa-engine/renovate/aws-java-sdk-monorepoHenning Baldersheim2024-02-211-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Update aws-java-sdk monorepo to v1.12.663
| * | | | | | Update aws-java-sdk monorepo to v1.12.663renovate[bot]2024-02-211-1/+1
| | | | | | |
* | | | | | | Merge pull request #30368 from ↵Henning Baldersheim2024-02-211-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/renovate/org.openrewrite.recipe-rewrite-testing-frameworks-2.x Update dependency org.openrewrite.recipe:rewrite-testing-frameworks to v2.4.1
| * | | | | | | Update dependency org.openrewrite.recipe:rewrite-testing-frameworks to v2.4.1renovate[bot]2024-02-211-1/+1
| |/ / / / / /
* | | | | | | Merge pull request #30371 from ↵Henning Baldersheim2024-02-211-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | vespa-engine/renovate/org.openrewrite.maven-rewrite-maven-plugin-5.x Update dependency org.openrewrite.maven:rewrite-maven-plugin to v5.23.1
| * | | | | | Update dependency org.openrewrite.maven:rewrite-maven-plugin to v5.23.1renovate[bot]2024-02-211-1/+1
| |/ / / / /
* | | | | | Merge pull request #30366 from ↵Henning Baldersheim2024-02-211-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/renovate/org.jetbrains.grammarkit-2022.x Update plugin org.jetbrains.grammarkit to v2022.3.2.2
| * | | | | | Update plugin org.jetbrains.grammarkit to v2022.3.2.2renovate[bot]2024-02-211-1/+1
| |/ / / / /
* | | | | | Merge pull request #30363 from vespa-engine/renovate/vite-5.x-lockfileHenning Baldersheim2024-02-211-3/+3
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Update dependency vite to v5.1.4
| * | | | | Update dependency vite to v5.1.4renovate[bot]2024-02-211-3/+3
| |/ / / /
* | | | | Merge pull request #30373 from ↵Geir Storli2024-02-216-15/+21
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/revert-30355-arnej/demand-newline-between-inputs Revert "demand newline between declarations of inputs"
| * | | | | Revert "demand newline between declarations of inputs"Harald Musum2024-02-216-15/+21
| |/ / / /
* | | | | Merge pull request #30365 from ↵Henning Baldersheim2024-02-212-1/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/renovate/github.com-klauspost-compress-1.x Update module github.com/klauspost/compress to v1.17.7
| * | | | | Update module github.com/klauspost/compress to v1.17.7renovate[bot]2024-02-212-1/+3
| |/ / / /
* | | | | Merge pull request #30362 from ↵Henning Baldersheim2024-02-211-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | vespa-engine/renovate/org.openrewrite.recipe-rewrite-recipe-bom-2.x Update dependency org.openrewrite.recipe:rewrite-recipe-bom to v2.7.1
| * | | | Update dependency org.openrewrite.recipe:rewrite-recipe-bom to v2.7.1renovate[bot]2024-02-211-1/+1
|/ / / /
* | | | Merge pull request #30356 from vespa-engine/hmusum/gc-unused-method-2Jon Marius Venstad2024-02-212-2/+0
|\ \ \ \ | | | | | | | | | | GC unused method
| * | | | GC unused methodHarald Musum2024-02-212-2/+0
| |/ / /
* | | | Merge pull request #30360 from vespa-engine/bratseth/classify-exceptionJon Bratseth2024-02-211-1/+1
|\ \ \ \ | |_|/ / |/| | | Should lead to 'application package', not 'internal' eror
| * | | Should lead to 'application package', not 'internal' erorJon Bratseth2024-02-211-1/+1
| | | |
* | | | Merge pull request #30300 from vespa-engine/vekterli/documentapi-over-protobufTor Brede Vekterli2024-02-21122-181/+4222
|\ \ \ \ | | | | | | | | | | Add new Protobuf-based MessageBus DocumentAPI protocol
| * | | | Tag Protobuf protocol boundary version and add binary test filesTor Brede Vekterli2024-02-2178-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All _reported_ versions >= 8.310 use Protobuf protocol, all lower versions use the legacy protocol. Reported version is controlled by an environment variable and defaults to 8.309, i.e. the legacy version.
| * | | | Improve codec error reporting and use Protobuf type parsers directlyTor Brede Vekterli2024-02-203-52/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Errors during (de-)serialization will now be logged, alongside the message type in question. Due to differences in internal wiring, the Java version logs the Document API type whereas C++ logs the protobuf message type (and also has throttled logging).
| * | | | Add new Protobuf-based MessageBus DocumentAPI protocolTor Brede Vekterli2024-02-1650-181/+4181
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds an entirely new implementation of the internal MessageBus DocumentAPI protocol, which shall be functionally 1-to-1 compatible with the existing legacy protocol. New protobuf schemas have been added to the top-level documentapi module, which are separated into different domains of responsibility: * CRUD messages * Visiting messages * Data inspection messages As well as a schema for shared, common message types. Both C++ and Java protocol implementations separate serialization and deserialization into a codec abstraction per message type, which hides the boilerplate required for Protobuf buffer management. The Java version is a tad more verbose due to generics type-erasure. This protocol does _not_ currently support lazy (de-)serialization in Java, as the existing mechanisms for doing so are inherently tied to the legacy protocol version. Performance tests will decide if we need to introduce such functionality to the new protocol version. To avoid having the new protocol go live in production, this commit changes the semantics of how MessageBus version reporting works (at least for the near future); instead of reporting the current Vespa _release_ version, it reports the highest supported _protocol_ version. This lets us conditionally enable the new protocol by reporting a MessageBus version greater than or equal to the protocol version _iff_ the protocol should be active. The new protocol is disabled by default. Other changes: * Protocol tests have been moved up one package directory level to be aligned with the actual package of the classes they test. This allows for using package-protected constructors in the serialization tests. * `DocumentDeserializer` now exposes the underlying document type repo/manager. This is done to detangle `Document`/`DocumentUpdate` deserialization from the underlying wire buffer management. * `RemoveLocationMessage` at long last contains a bucket space, which was forgotten when we initially added this concept to the other messages, and where the pain of adding it in later was too big (not so anymore!). Unit tests for both C++ and Java have been hoisted from the legacy test suite, cleaned up and extended with additional cases. The C++ tests use the old unit test kit and should receive a good follow-up washing and GTest-rewrite. **Important**: due to how MessageBus protocol versioning works, the final protocol version is _not_ yet decided, as setting it requires syncing against our build systems. A follow-up commit will assign the final version as well as include all required binary test files.
* | | | | Merge pull request #30355 from vespa-engine/arnej/demand-newline-between-inputsArne H Juul2024-02-216-21/+15
|\ \ \ \ \ | |_|_|/ / |/| | | | demand newline between declarations of inputs
| * | | | demand newline between declarations of inputsArne Juul2024-02-216-21/+15
| | | | |
* | | | | Merge pull request #30353 from ↵v8.308.26Henning Baldersheim2024-02-201-0/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/toregge/propagate-omit-summary-features-from-summary-result-class-to-docsum-state-in-streaming-mode Propagate omit summary features from summary result class to
| * | | | | Propagate omit summary features from summary result class toTor Egge2024-02-201-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | docsum state in streaming mode.
* | | | | | Merge pull request #30354 from ↵Harald Musum2024-02-204-7/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | vespa-engine/revert-30348-renovate/commons-compress.vespa.version Revert "Update dependency org.apache.commons:commons-compress to v1.26.0" MERGEOK
| * | | | | Revert "Update dependency org.apache.commons:commons-compress to v1.26.0"Henning Baldersheim2024-02-204-7/+3
|/ / / / /
* | | | | Merge pull request #30348 from ↵Henning Baldersheim2024-02-204-3/+7
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/renovate/commons-compress.vespa.version Update dependency org.apache.commons:commons-compress to v1.26.0