[ 
https://issues.apache.org/jira/browse/CALCITE-4737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17399384#comment-17399384
 ] 

Zuozhi Wang commented on CALCITE-4737:
--------------------------------------

I cleaned up the visulaizer and removed the TVR-related code. Here's the pull 
request: https://github.com/apache/calcite/pull/2486   Reviews and suggestions 
are all welcomed.

Here's a couple more issues to discuss:
1. What's a good way of including the visualizer code into VolcanoPlanner and 
let developers use it easily? One way is to create the visualizer and the event 
listener inside `VolcanoPlanner.findBestExp`, and introduce a variable 
somewhere to turn it on/off. This visualizer might slow down optimization quite 
a bit for large sized plans.

2. Where's a good place to put the frontend HTML files? The java file need to 
find the HTML template. Right now they are under "core/src/main/resources". 
Normal test cases under "core" project runs fine, but if it's run outside of 
"core", I'm not sure if it's able to find the resource.

> Add Volcano visualizer for debugging
> ------------------------------------
>
>                 Key: CALCITE-4737
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4737
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Julian Hyde
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Add Volcano visualizer for debugging.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to