Github user rdblue commented on the issue:

    https://github.com/apache/spark/pull/20692
  
    I don't think it is a good idea to block a fix like this on a redesign of 
explain plans. If that redesign was currently being worked on and this made it 
more difficult, it might be reasonable. But this can be a small interim fix and 
doesn't really affect larger plans.
    
    If this patch needs to be fixed to be more clean, then let's get that 
feedback to Marco. He has been really responsive on this contribution.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to