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

Update dependencies #18232

Merged
merged 10 commits into from
Jul 15, 2023
Merged

Update dependencies #18232

merged 10 commits into from
Jul 15, 2023

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Jul 11, 2023

I want to do it early in the release cycle to detect regressions as fast as possible

The only functional change is that BigQuery arrow implementation no longer requires --add-opens JVM flags, which is reflected in the documentation.

@github-actions github-actions bot added docs bigquery BigQuery connector labels Jul 11, 2023
@wendigo wendigo force-pushed the serafin/dependency-updates branch from 7e2d747 to b09f602 Compare July 12, 2023 04:26
@wendigo
Copy link
Contributor Author

wendigo commented Jul 12, 2023

/test-with-secrets sha=b09f60299320edca03a618495dd1abad4b9c5d4a

@wendigo wendigo force-pushed the serafin/dependency-updates branch from b09f602 to 835f4d4 Compare July 13, 2023 16:04
@wendigo wendigo requested review from electrum, martint and ebyhr July 14, 2023 08:37
@wendigo
Copy link
Contributor Author

wendigo commented Jul 14, 2023

Unfortunately Arrow does not work without add-opens even for 12.0.1 :(

@wendigo wendigo requested a review from kokosing July 14, 2023 12:20
@wendigo
Copy link
Contributor Author

wendigo commented Jul 14, 2023

/test-with-secrets sha=835f4d4774db91653340b3d6d7144fe4a1ff6511

@wendigo wendigo merged commit 0981eac into trinodb:master Jul 15, 2023
@wendigo wendigo deleted the serafin/dependency-updates branch July 15, 2023 10:50
@github-actions github-actions bot added this to the 423 milestone Jul 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants