[GitHub] [spark] rxin commented on issue #23811: [SPARK-26902][SQL] Support java.time.Instant as an external type of TimestampType

2020-01-29 Thread GitBox
rxin commented on issue #23811: [SPARK-26902][SQL] Support java.time.Instant as an external type of TimestampType URL: https://github.com/apache/spark/pull/23811#issuecomment-579939604 But if it is off by default, it means almost nobody will be using the new type right? My point is that

[GitHub] [spark] rxin commented on issue #23811: [SPARK-26902][SQL] Support java.time.Instant as an external type of TimestampType

2020-01-24 Thread GitBox
rxin commented on issue #23811: [SPARK-26902][SQL] Support java.time.Instant as an external type of TimestampType URL: https://github.com/apache/spark/pull/23811#issuecomment-578335396 How useful is this change? Wouldn't it break a lot of user code that use Timestmap type when they