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 #402

Merged
merged 11 commits into from
Mar 18, 2024
Merged

Update dependencies #402

merged 11 commits into from
Mar 18, 2024

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Mar 17, 2024

No description provided.

@wendigo
Copy link
Contributor Author

wendigo commented Mar 17, 2024

PTAL @electrum. This makes it possible to build with target language level 22

@wendigo
Copy link
Contributor Author

wendigo commented Mar 18, 2024

@electrum @martint PTAL. I'd like to update it in Trino and test with JDK 22-ea

@wendigo
Copy link
Contributor Author

wendigo commented Mar 18, 2024

I've tested these changes with Airlift and Trino and seems fine (code needs some fixes due to deprecated guava's Throwables methods but these are minor)

@martint martint merged commit 9cf3b20 into airlift:master Mar 18, 2024
2 checks passed
@wendigo wendigo deleted the serafin/updates-mar-17 branch March 18, 2024 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants