[CALCITE-7427] Add connection config to enable RuleMatchVisualizer
Quality Gate passed
Issues
5 New issues
0 Accepted issues
Measures
0 Security Hotspots
77.4% Coverage on New Code
0.0% Duplication on New Code
@tjbanghart, thank you for the PR! It makes it easier to use the RuleMatchVisualizer. When I use the visualizer, I always have to add the same code: an AtomicInteger to distinguish the queries and some code to define the output. I would appreciate to get it into Calcite.
Thank you both for the reviews! I'll be able to revisit shortly.
@thomasrebele since you're here, is there a way to highlight the cheapest path in the visualizer? If not is there an existing issue to add this functionality?
@tjbanghart, could you explain what you mean by the cheapest path?
I've got a draft to improve the visualizer. You can have a look at https://github.com/apache/calcite/pull/4506. If you have ideas for improvement, feel free to add a comment to https://issues.apache.org/jira/browse/CALCITE-7133 or create a new ticket and tag me in a comment.
The improvement has a feature to show a sub-graph connecting some specified nodes (and hiding all other nodes). Maybe that's close enough to your use case?
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.