Bug fix/traversal collection detection fix #10801
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
Simran-B
reviewed
Jan 8, 2020
|
Unrelated Jenkins failure. |
hkernbach
commented
Jan 8, 2020
|
Tests blue. |
mchacki
requested changes
Jan 9, 2020
…versal-collection-detection-fix
mchacki
requested changes
Jan 13, 2020
|
Sorry, added. |
…versal-collection-detection-fix
|
Jenkins green! |
ObiWahn
added a commit
that referenced
this pull request
Jan 16, 2020
…-fix/speedup-aql-datetime-parsing-pegtl * 'devel' of https://github.com/arangodb/arangodb: Bug fix/windows exe product info (#10869) Bug fix/traversal collection detection fix (#10801) fix compilation error Bug fix/issue 10852 (#10865) Update CHANGELOG apply documentation fix from @Simran-B
ObiWahn
added a commit
that referenced
this pull request
Jan 16, 2020
…ture/mimalloc * 'devel' of https://github.com/arangodb/arangodb: (636 commits) Bug fix/windows exe product info (#10869) Bug fix/traversal collection detection fix (#10801) fix compilation error Bug fix/issue 10852 (#10865) Update CHANGELOG apply documentation fix from @Simran-B Update CHANGELOG remove remainders of allowDirectExecution (#10872) naively implement timeout for waitpid (#9943) Fix string comparison for AstNodes (#10837) Mention Web UI and command-line client tools in README.md (#10790) Bug fix/fixing multiple minor ui issues (#10700) Feature/http nextgen refactor (#10851) upgrade velocypack library (#10758) Remove uses of static ApplicationServer (#10806) speed up datetime parsing in AQL (#10846) make the logging a bit more efficient (#10849) switch log-file location depending on all/user installation, remove on uninstallation (#10832) changed roundCosts to removeCosts (#10638) fixed issue 10725 (#10750) ...
ObiWahn
added a commit
that referenced
this pull request
Jan 17, 2020
…ture/insert-or-update * 'devel' of https://github.com/arangodb/arangodb: (27 commits) fix edge case with padding in VelocyPack arrays (#10901) Bug fix/windows exe product info (#10869) Bug fix/traversal collection detection fix (#10801) fix compilation error Bug fix/issue 10852 (#10865) Update CHANGELOG apply documentation fix from @Simran-B Update CHANGELOG remove remainders of allowDirectExecution (#10872) naively implement timeout for waitpid (#9943) Fix string comparison for AstNodes (#10837) Mention Web UI and command-line client tools in README.md (#10790) Bug fix/fixing multiple minor ui issues (#10700) Feature/http nextgen refactor (#10851) upgrade velocypack library (#10758) Remove uses of static ApplicationServer (#10806) speed up datetime parsing in AQL (#10846) make the logging a bit more efficient (#10849) switch log-file location depending on all/user installation, remove on uninstallation (#10832) changed roundCosts to removeCosts (#10638) ...
ObiWahn
added a commit
that referenced
this pull request
Jan 17, 2020
…ture/insert-or-update * 'devel' of https://github.com/arangodb/arangodb: (27 commits) fix edge case with padding in VelocyPack arrays (#10901) Bug fix/windows exe product info (#10869) Bug fix/traversal collection detection fix (#10801) fix compilation error Bug fix/issue 10852 (#10865) Update CHANGELOG apply documentation fix from @Simran-B Update CHANGELOG remove remainders of allowDirectExecution (#10872) naively implement timeout for waitpid (#9943) Fix string comparison for AstNodes (#10837) Mention Web UI and command-line client tools in README.md (#10790) Bug fix/fixing multiple minor ui issues (#10700) Feature/http nextgen refactor (#10851) upgrade velocypack library (#10758) Remove uses of static ApplicationServer (#10806) speed up datetime parsing in AQL (#10846) make the logging a bit more efficient (#10849) switch log-file location depending on all/user installation, remove on uninstallation (#10832) changed roundCosts to removeCosts (#10638) ...
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.
Pull Request Guidelines
Scope & Purpose
An internal map was not filled with edge collections, where they should be. It was forgotten that edges can also point to a different edge and not only vertices.
Devel and 3.6.0 needs to be fixed.
Testing & Verification
This PR adds tests that were used to verify all changes: