planga82 commented on issue #23067: [SPARK-26097][Web UI] Add the new 
partitioning description to the exchange node name
URL: https://github.com/apache/spark/pull/23067#issuecomment-451681382
 
 
   I'm not sure about this pull request. In the DAG visualization there are 
other types of nodes, now all types of nodes has coherent information, only the 
name. I think your proposal is interesting but with a deeper change. We could 
think what new information is interesting to visualice in the DAG and apply it 
to all different type of nodes (not only one type of exchange node).
   
   The new information in the nodes i think that needs to be more intuitive for 
the users.
   
   The last thing, 'im not sure where could impact this change of name node in 
other parts of the code

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to