[GitHub] [incubator-iceberg] electrum commented on issue #279: Transforming timestamp to date should produce date

2019-08-08 Thread GitBox
electrum commented on issue #279: Transforming timestamp to date should produce 
date
URL: 
https://github.com/apache/incubator-iceberg/issues/279#issuecomment-519572394
 
 
   Use `java.time.LocalDate`. The older `java.util.Date` actually represents an 
instant.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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



[GitHub] [incubator-iceberg] electrum commented on issue #279: Transforming timestamp to date should produce date

2019-07-17 Thread GitBox
electrum commented on issue #279: Transforming timestamp to date should produce 
date
URL: 
https://github.com/apache/incubator-iceberg/issues/279#issuecomment-512439770
 
 
   Should `day` also not support the `date` type, since that would be an 
identity transform?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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