Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/16331
  
    I don't think this can be the right change -- it's construing any date 
before 2000 as invalid, which may happen to be OK but is pretty arbitrary. It 
arises as the result of formatting -1 as a date, and this method seems to try 
to handle that, but it's comparing the string to -1 I guess. Surely this should 
be handled elsewhere, like in whatever calls `formatDate`? 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to