trino
trino copied to clipboard
Use Aircompressor in Avro library code
Description
Substitute Aircompressor code for Avro Codec Factories for snappy and Zstd.
Additional context and related issues
Allows us to further carve out Hadoop dependencies.
Release notes
(X ) This is not user-visible or docs only and no release notes are required. ( ) Release notes are required, please propose a release note for me. ( ) Release notes are required, with the following suggested text:
# Section
* Fix some things. ({issue}`issuenumber`)
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua
:wave: @jklamer - this PR has become inactive. We hope you are still interested in working on it. Please let us know, and we can try to get reviewers to help with that.
We're working on closing out old and inactive PRs, so if you're too busy or this has too many merge conflicts to be worth picking back up, we'll be making another pass to close it out in a few weeks.
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua
Is this still in progress @jklamer .. also @electrum is this still something we need or was this removal already done separately?
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua
Closing this pull request, as it has been stale for six weeks. Feel free to re-open at any time.
I think this is still valid and in progress @jklamer @electrum
I'll defer to @electrum. IIRC we wanted this change to avoid any sort of native code memory leaks for catalogConnector lifecycles. I know there is a concern that this Java version is noticeably slower.
We still want to do this. Now that Java 22 has improved support for native code and fixed the GC locker issue, we plan to use native libraries in Aircompressor. Centralizing usage will let us upgrade and manage everything in one place.
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua
Added stale-ignore so that @jklamer and @electrum can proceed without the PR getting closed.