Feature/hybrid smart graph test setup helper #14455
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…s is mainly used in Enterprise Code
…at the relevant position
…te correct smartgraph schema
…smart-edge-collections
…godb into feature/hybrid-smart-graph-testsuite
…thub.com:arangodb/arangodb into feature/hybrid-smart-graph-testsuite
…godb into feature/hybrid-smart-graph-testsuite
…godb into feature/hybrid-smart-graph-testsuite
…red for SmartGraphs
…esult instead - will lead to more flexibility handling the result
…riant, we need to modify the result to have it correct
…se them from different places as long as we do not do it in multithreaded fashion
…ngodb/arangodb into feature/hybrid-smart-graph-testsuite
…godb into feature/hybrid-smart-graph-testsuite
…ngodb/arangodb into feature/hybrid-smart-graph-testsuite
… check for the data
…, it cannot decide this
…es. It should not matter in Production, but is callable from within the tests
… to get TraversalOptions ready to plan correct idnexes
…wn a mock coordinator
… feature/hybrid-smart-graph-test-setup-helper
hkernbach
requested changes
Jul 6, 2021
| @@ -32,6 +32,7 @@ | |||
| #include "Rest/GeneralResponse.h" | |||
| #include "Transaction/StandaloneContext.h" | |||
|
|
|||
| #include <Logger/LogMacros.h> | |||
There was a problem hiding this comment.
Think this can be removed now.
Co-authored-by: Heiko <hkernbach@users.noreply.github.com>
…raph-test-setup-helper
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Intermediate Merge.
This improves the MockServers, to support Indexes on Coordinators.
For some reason activating this causes some tests to get into assertions, so we made it explicit to activate the ClusterIndexes.
This assertion can be solved by "Mocking" the creation on MockStorageEngine as well, and not reuse the Production index Code