ambition119 commented on issue #96: support from unix timestamp to string date
URL: https://github.com/apache/calcite-avatica/pull/96#issuecomment-493292562
 
 
   > I'm open to including these changes in principle.
   > 
   > However, misunderstanding the difference between moments (e.g. millis 
since 1970 UTC) and zoneless timestamps (e.g. millis since zoneless 1970) is a 
major source of bugs. Especially as values flow between Java and SQL and back 
again. Avatica has to be 100% clear and consistent.
   > 
   > Your PR, as it stands, is confused on this issue.
   
   My code is only available with a specific scope, thank you for your 
discussion and guidance.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to