ARROW-977: [java] Add Timezone aware timestamp vectors #644
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
julienledem
changed the title
Change-Id: Iad4f080581382fa95a76cb5f64204111ead09a3d
Change-Id: I6960a9725489d4e9e396ac352d37d731e48de75e
Change-Id: Ibbe3453e3b627b5187c496d91d296e538dd3740d
Change-Id: I060ef92dbb002202cb964280a122a2f4f3ad6fdb
|
@wesm rebased |
wesm
approved these changes
May 13, 2017
julienledem
changed the title
|
@wesm: fixed. Sorry about that |
|
Since we now have 8 timestamp classes in Java, if there is any way to simplify in the near future to limit the collateral damage in downstream projects of future API changes, that would be great |
jeffknupp
pushed a commit
to jeffknupp/arrow
that referenced
this pull request
Jun 3, 2017
Author: Julien Le Dem <julien@apache.org> Closes apache#644 from julienledem/TZ and squashes the following commits: 37987b9 [Julien Le Dem] add integration tests a58fcae [Julien Le Dem] add integration test 39966aa [Julien Le Dem] add other vectors and tests bf245ce [Julien Le Dem] add TZ vectors
pcmoritz
pushed a commit
to pcmoritz/arrow
that referenced
this pull request
Jun 11, 2017
Author: Julien Le Dem <julien@apache.org> Closes apache#644 from julienledem/TZ and squashes the following commits: 37987b9 [Julien Le Dem] add integration tests a58fcae [Julien Le Dem] add integration test 39966aa [Julien Le Dem] add other vectors and tests bf245ce [Julien Le Dem] add TZ vectors
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.