milenkovicm commented on PR #1436:
URL: 
https://github.com/apache/datafusion-ballista/pull/1436#issuecomment-4136548418

   this is getting better and better @martin-g, cool work ! 
   
   If i can suggest, 
   
   <img width="1724" height="409" alt="image" 
src="https://github.com/user-attachments/assets/5067b120-fd60-47f8-aa7e-2373998b80ce";
 />
   
   would it make sense to have only ability to switch between plans, show stage 
plan as default but give option to look at physical plan and logical plan.  I 
believe current setup will get too busy quite quickly. to demonstrate how busy 
it can get just run tps-h benchmark
   
   <img width="1716" height="782" alt="image" 
src="https://github.com/user-attachments/assets/c2d9bebc-2d9d-42c4-9d26-f9f760d6b226";
 />
   
   
   ```
   cargo run --bin tpch -- benchmark ballista -p 
/Users/.../TMP/tpch_data/tpch-data-sf1/ -f parquet -i 1 --port 50050 --host 
127.0.0.1 -q 2
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to