Github user koeninger commented on the issue:

    https://github.com/apache/spark/pull/19789
  
    What are you actually asserting in that test and/or does it reliably fail
    if run on the version of your code before the patch?
    
    On Wed, Nov 22, 2017 at 1:33 PM, Daroo <notificati...@github.com> wrote:
    
    > I've added a test. @koeninger <https://github.com/koeninger> is it
    > something you had in mind?
    >
    > —
    > You are receiving this because you were mentioned.
    > Reply to this email directly, view it on GitHub
    > <https://github.com/apache/spark/pull/19789#issuecomment-346452116>, or 
mute
    > the thread
    > 
<https://github.com/notifications/unsubscribe-auth/AAGAB_Sz5wdH2nf4eM0z-DMwAInQD0MQks5s5HclgaJpZM4QklrV>
    > .
    >



---

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

Reply via email to