Skip to content
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

LS performance: make BinaryTransitiveClosureCheck work in either direction #122

Open
eclipse-viatra-bot opened this issue Mar 12, 2024 · 0 comments
Labels
bugzilla Issues migrated from Eclipse bugzilla. Query Issues related to the query component of VIATRA, including runtime or pattern language issues.

Comments

@eclipse-viatra-bot
Copy link

| --- | --- |
| Bugzilla Link | 563225 |
| Status | NEW |
| Importance | P3 enhancement |
| Reported | May 15, 2020 11:50 EDT |
| Modified | May 15, 2020 11:50 EDT |
| Version | 2.3.0 |
| Reporter | Gabor Bergmann |

Description

BinaryTransitiveClosureCheck operations in the LS backend (i.e. checking find foo+(x,y) whan x and y are known already) have suboptimal performance.

Right now, the executor only traverses from source to (transitive) targets, even though the other direction is just as feasible. There are cases where target-to-source would be preferable due to lower fan-outs, especially if each target has only one associated source (typical with transitive containment check). The planner should be able to choose the more favorable direction.

@eclipse-viatra-bot eclipse-viatra-bot added bugzilla Issues migrated from Eclipse bugzilla. legacy Query Issues related to the query component of VIATRA, including runtime or pattern language issues. labels Mar 12, 2024
@ujhelyiz ujhelyiz removed the legacy label Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from Eclipse bugzilla. Query Issues related to the query component of VIATRA, including runtime or pattern language issues.
Projects
None yet
Development

No branches or pull requests

2 participants