Extensible query planner & relation evaluation fixes #139 #147
+197
−93
Conversation
This file contains 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
* The Rete recipe compiler is now extensible through the use of ExternalQueryPlanProvider. * JoinCandidate visibility is changed, so that ExternalQueryPlanProviders can reuse the class. * The communication tracker correctly considers the dependencies between relation evaluator nodes, batching receivers, and the batching receivers' transitive sources. * Small performance improvement in RelationEvaluatorNode to guard against spurious batchUpdateCompleted calls as much as possible. Signed-off-by: Tamas Szabo <szabta89@github.com> Change-Id: I948e80a3057f63416a78c072073107960678adeb
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.
PR was initialized from https://git.eclipse.org/r/c/viatra/org.eclipse.viatra/+/197358, rebased over current master branch.
@szabta89, @bergmanngabor Please verify that everything is as needed and if there are discussions in the Gerrit change that is necessary to be kept indefinitely please copy it into this PR. Thanks.