Skip to content
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

Bump to latest arrow release #5947

Open
devinrsmith opened this issue Aug 15, 2024 · 3 comments
Open

Bump to latest arrow release #5947

devinrsmith opened this issue Aug 15, 2024 · 3 comments
Assignees
Labels
Milestone

Comments

@devinrsmith
Copy link
Member

devinrsmith commented Aug 15, 2024

We are currently on arrow version 13.0.0; the latest is 17.0.0. https://arrow.apache.org/release/

During a quick test to see if 17.0.0 arrow FlightSql JDBC drivers fixed an issue, the server was unable to compile, I think due to flight-grpc version not available. (Note: I could create a new project to test the client in isolation, so bumping arrow versions on the server is not a technical blocker to getting a newer standalone client up... it just takes a bit more work to stand up that client.) Something may need changing wrt https://repo.maven.apache.org/maven2/org/apache/arrow/flight-grpc/; it seems to either have a different release cadence than arrow, or it has changed to a new groupId or name, or is no longer necessary.

Trying 15.0.2 (the last version where arrow flight-grpc was "in-sync") resulted in compiler errors, potentially related to the gRPC version. As such, this issue may be dependent on #5299.

@devinrsmith devinrsmith added feature request New feature or request version-bump labels Aug 15, 2024
@devinrsmith devinrsmith added this to the 5. Backlog milestone Aug 15, 2024
@rcaudy rcaudy modified the milestones: 5. Backlog, 0.37.0 Aug 15, 2024
@devinrsmith
Copy link
Member Author

From the perspective of FlightSQL JDBC testing, it would also be nice to bump to pick up the logging fix apache/arrow#40908

@nbauernfeind
Copy link
Member

Be sure to ping me to add a commit either to the upgrade PR or a follow up after the upgrade, to finish the ListView support I've added in my barrage type-mapping refactorings (PR not yet uploaded). I have everything except to check for the ArrowTypeID which doesn't exist in our current version.

@devinrsmith
Copy link
Member Author

It's also more than a logging fix; it looks like older versions of the FlightSQL JDBC driver treat executeQuery("SELECT 1") as an UPDATE... which is pretty preposterous, but that seems to have been fixed in newer versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants