Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/5984#issuecomment-104426757
Let me take another look in a little bit. There's been a bit of debate
over whether we want to commit to supporting 128-bit timestamps in Tungsten or
restrict their precision to 64 bits. The decision that we make for 1.4 may not
have to be final, though, since this is an experimental feature that's off by
default. Since this patch seems like a clear improvement to the current code
and since we'll have the flexibility to change this later, I'm inclined to
merge this for 1.4.0 and address the timestamp questions in more detail for
1.5.0.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
