-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-400: set struct length on load #234
Closed
Closed
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
Change-Id: I9b92eaf56f5a9b5b486eec92356b2fc1b84755cb
Build is green save for OS X. +1 |
wesm
pushed a commit
to wesm/arrow
that referenced
this pull request
Sep 2, 2018
This patch allows to port parquet-cpp on non-intel architectures and/or linux/unix flavors where CPU info is different or not available. Tested on Alpinelinux (musl libc). Author: Marc Vertes <mvertes@free.fr> Closes apache#234 from mvertes/master and squashes the following commits: 1b087be [Marc Vertes] PARQUET-862: Provide defaut cache size values
wesm
pushed a commit
to wesm/arrow
that referenced
this pull request
Sep 4, 2018
This patch allows to port parquet-cpp on non-intel architectures and/or linux/unix flavors where CPU info is different or not available. Tested on Alpinelinux (musl libc). Author: Marc Vertes <mvertes@free.fr> Closes apache#234 from mvertes/master and squashes the following commits: 1b087be [Marc Vertes] PARQUET-862: Provide defaut cache size values Change-Id: I600915833d8f942f96e2cf38240493709c95b28c
wesm
pushed a commit
to wesm/arrow
that referenced
this pull request
Sep 6, 2018
This patch allows to port parquet-cpp on non-intel architectures and/or linux/unix flavors where CPU info is different or not available. Tested on Alpinelinux (musl libc). Author: Marc Vertes <mvertes@free.fr> Closes apache#234 from mvertes/master and squashes the following commits: 1b087be [Marc Vertes] PARQUET-862: Provide defaut cache size values Change-Id: I600915833d8f942f96e2cf38240493709c95b28c
wesm
pushed a commit
to wesm/arrow
that referenced
this pull request
Sep 7, 2018
This patch allows to port parquet-cpp on non-intel architectures and/or linux/unix flavors where CPU info is different or not available. Tested on Alpinelinux (musl libc). Author: Marc Vertes <mvertes@free.fr> Closes apache#234 from mvertes/master and squashes the following commits: 1b087be [Marc Vertes] PARQUET-862: Provide defaut cache size values Change-Id: I600915833d8f942f96e2cf38240493709c95b28c
wesm
pushed a commit
to wesm/arrow
that referenced
this pull request
Sep 8, 2018
This patch allows to port parquet-cpp on non-intel architectures and/or linux/unix flavors where CPU info is different or not available. Tested on Alpinelinux (musl libc). Author: Marc Vertes <mvertes@free.fr> Closes apache#234 from mvertes/master and squashes the following commits: 1b087be [Marc Vertes] PARQUET-862: Provide defaut cache size values Change-Id: I600915833d8f942f96e2cf38240493709c95b28c
rafael-telles
added a commit
to rafael-telles/arrow
that referenced
this pull request
Jan 25, 2022
* Set netty property when JDBC driver is loaded (apache#235) Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> * [Java] [JDBC] Change the statement implementation to reduce the job numbers. (apache#234) * Properly read incoming PreparedStatement columnmetadata * Make map Thread-safe, rename map and change getPreparedStatement return type Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> * Add column metadata when creating PreparedStatement (apache#233) Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> * Minor fixes Co-authored-by: Kyle Porter <kporter@dremio.com> Co-authored-by: Jose Almeida <almeidajcr90@gmail.com> Co-authored-by: Rafael Telles <rafael@telles.dev> Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> Co-authored-by: kylep-dremio <38920967+kylep-dremio@users.noreply.github.com> Co-authored-by: iurysalino <iurysalino@gmail.com> Co-authored-by: Gabriel Escobar <51451696+escobargabriel@users.noreply.github.com>
vfraga
added a commit
to rafael-telles/arrow
that referenced
this pull request
Mar 29, 2022
* Set netty property when JDBC driver is loaded (apache#235) Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> * [Java] [JDBC] Change the statement implementation to reduce the job numbers. (apache#234) * Properly read incoming PreparedStatement columnmetadata * Make map Thread-safe, rename map and change getPreparedStatement return type Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> * Add column metadata when creating PreparedStatement (apache#233) Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> * Minor fixes Co-authored-by: Kyle Porter <kporter@dremio.com> Co-authored-by: Jose Almeida <almeidajcr90@gmail.com> Co-authored-by: Rafael Telles <rafael@telles.dev> Co-authored-by: Vinicius Fraga <sxvinifp@gmail.com> Co-authored-by: kylep-dremio <38920967+kylep-dremio@users.noreply.github.com> Co-authored-by: iurysalino <iurysalino@gmail.com> Co-authored-by: Gabriel Escobar <51451696+escobargabriel@users.noreply.github.com>
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.
Adds unit test, closes #233