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

   thanks @martin-g
   I think stage view makes sense, this is super useful 
   
   stage view looks ok, its easy to get to, just few little things to declutter 
it a bit 
   
   <img width="1343" height="669" alt="image" 
src="https://github.com/user-attachments/assets/ab841f49-b278-4b35-9890-56215c94bb71";
 />
   
   - instead of showing number of input/output rows have like 17392641 -> 17.3M 
would it be possible (we have ns, ms for compute time)?
   
   - also task duration, can we change from `min/med/max/p25/p75` -> 
`min/p25/med/p75/max` and from `min=1 med=1, max =150, p25=1, p75=1` -> to 
`1/1/1/1/150` it looks a bit busy, and if we remove `min=` we could probably 
fit input rows distribution 
   
   for task id:
   
   <img width="1642" height="519" alt="image" 
src="https://github.com/user-attachments/assets/a41e7a02-febc-4b93-a209-d21e53e7f407";
 />
   
   - title is broken, stage id is missing
   - a bit of visual separation  between once this is opened would make sense, 
perhaps a smaller pop up box, so users could understand its a new box 
   
   
   let me know what you think 


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