calcite icon indicating copy to clipboard operation
calcite copied to clipboard

[CALCITE-7427] Add connection config to enable RuleMatchVisualizer

Open tjbanghart opened this issue 2 months ago • 5 comments

CALCITE-7247

tjbanghart avatar Oct 25 '25 20:10 tjbanghart

@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.

thomasrebele avatar Nov 03 '25 15:11 thomasrebele

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 avatar Nov 04 '25 16:11 tjbanghart

@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?

thomasrebele avatar Nov 05 '25 09:11 thomasrebele

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.

github-actions[bot] avatar Dec 06 '25 03:12 github-actions[bot]