Github user JoshRosen commented on the issue:

    https://github.com/apache/spark/pull/21346
  
    I'm going to be starting a more detailed review pass on this now and will 
be getting caught back up with the discussion that's happened so far.
    
    One high-level point I'd like to keep in mind: what are the major risks of 
this change in terms of introducing performance or correctness issues? If we 
identify risks (e.g. "this is a historically tricky area of code?"), can we 
mitigate those risks through correctness testing /  load testing?


---

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

Reply via email to