You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "vlad-lyutenko (via GitHub)" <gi...@apache.org> on 2025/03/26 10:23:13 UTC
[PR] Add SecurityManagerCompatibility shim [kudu]
vlad-lyutenko opened a new pull request, #63:
URL: https://github.com/apache/kudu/pull/63
This is a shim to allow the use of both modern (JRE 18+) and legacy APIs (deprecated in 17, degraded in 23, removal unknown) related to [JEP 411: Deprecate the Security Manager for Removal](https://openjdk.org/jeps/411).
This shim implements the interim strategy outlined in [KIP-1006: Remove SecurityManager Support](https://cwiki.apache.org/confluence/display/KAFKA/KIP-1006%3A+Remove+SecurityManager+Support) where legacy APIs are used as long as they are available and functional. Once the legacy APIs are removed or degraded, the modern APIs are used instead.
The shim is structured as a general interface with four 'strategy' implementations for testability. These implementations allow for mocking out the classloading infrastructure to simulate situations which no current JRE implements, namely removal and further degradation of functionality
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "zchovan (via GitHub)" <gi...@apache.org>.
zchovan closed pull request #63: Add SecurityManagerCompatibility shim
URL: https://github.com/apache/kudu/pull/63
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "alexeyserbin (via GitHub)" <gi...@apache.org>.
alexeyserbin commented on PR #63:
URL: https://github.com/apache/kudu/pull/63#issuecomment-2755190844
Hi @vlad-lyutenko, thank you very much for this PR!
As @zchovan mentioned, the Kudu project doesn't merge new code through PRs; we use the [gerrit](https://www.gerritcodereview.com/) tool to do code review and submission. Could you follow the guidance [here](https://kudu.apache.org/docs/contributing.html) to adapt this PR into a gerrit change?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "zchovan (via GitHub)" <gi...@apache.org>.
zchovan commented on PR #63:
URL: https://github.com/apache/kudu/pull/63#issuecomment-2754552996
@vlad-lyutenko the Kudu community doesn't accept code contributions via pull requests, the changes are reviewed and merged via gerrit: https://gerrit.cloudera.org/q/status:open+project:kudu
I'd recommend submitting this patch there. Thanks for looking into this!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "zchovan (via GitHub)" <gi...@apache.org>.
zchovan commented on PR #63:
URL: https://github.com/apache/kudu/pull/63#issuecomment-3714573137
This change was merged via Gerrit:
https://gerrit.cloudera.org/c/22703/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "hqbhoho (via GitHub)" <gi...@apache.org>.
hqbhoho commented on PR #63:
URL: https://github.com/apache/kudu/pull/63#issuecomment-3435569547
Hi, how is the progress of this work?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "martongreber (via GitHub)" <gi...@apache.org>.
martongreber commented on PR #63:
URL: https://github.com/apache/kudu/pull/63#issuecomment-3442426598
> Hi, how is the progress of this work?
Hi!
The change has been review and merged: https://gerrit.cloudera.org/#/c/22703/
Merged commit:
https://github.com/apache/kudu/commit/586cfc20905df35f49541b53af4442736790b0c1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add SecurityManagerCompatibility shim [kudu]
Posted by "vlad-lyutenko (via GitHub)" <gi...@apache.org>.
vlad-lyutenko commented on PR #63:
URL: https://github.com/apache/kudu/pull/63#issuecomment-2757765676
Sure, just made it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@kudu.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org