8000 Restrict hybrid disjoint sg directions by mchacki · Pull Request #16214 · arangodb/arangodb · GitHub
[go: up one dir, main page]

Skip to content

Restrict hybrid disjoint sg directions #16214

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
May 24, 2022

Conversation

mchacki
Copy link
Member
@mchacki mchacki commented May 2, 2022

Scope & Purpose

This PR hits the enterprise version only.
For Hybrid Disjoint SmartGraphs we decided to restrict switches between Smart and Satellite
for a single AQL Node to at most one switch.
So every Graph node that switches two (or more) times will cause the query to reject.
Note: This is a breaking change! there can be queries that do not work anymore.
However original behavior especially for path queries, delivered undefined results.

Please see Enterprise PR for technical details.

  • 💩 Bugfix
  • 🍕 New feature
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification

Checklist

  • Tests
    • Regression tests
    • C++ Unit tests
    • integration tests
    • resilience tests
  • 📖 CHANGELOG entry made
  • 📚 documentation written (release notes, API changes, ...)
  • Backports
    • Backport for 3.9: (Please link PR)
    • Backport for 3.8: (Please link PR)
    • Backport for 3.7: (Please link PR)

Related Information

(Please reference tickets / specification / other PRs etc)

@mchacki mchacki added this to the devel milestone May 2, 2022
@mchacki mchacki self-assigned this May 2, 2022
@mchacki mchacki requested a review from a team as a code owner May 2, 2022 14:46
@mchacki mchacki requested a review from goedderz May 2, 2022 14:47
@mchacki mchacki merged commit 34ff094 into devel May 24, 2022
@mchacki mchacki deleted the bug-fix/restrict-hybrid-disjoint-sg-directions branch May 24, 2022 14:29
mchacki added a commit that referenced this pull request May 25, 2022
@KVS85 KVS85 added 1 Bug 3 Graph Graph query engine labels Jun 3, 2022
KVS85 added a commit that referenced this pull request Jun 7, 2022
* Restrict hybrid disjoint sg directions (#16214)

* Make VPack assertion more relaxed, it should not trigger if the number is stored in a different number type.

* Added generated files

* CLang formated generated files

* Apply suggestions from code review

Co-authored-by: Jan <jsteemann@users.noreply.github.com>

* Applied review requests

* Update CHANGELOG

Co-authored-by: Jan <jsteemann@users.noreply.github.com>
Co-authored-by: Vadim <vadim@arangodb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 Bug 3 Graph Graph query engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0