You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Andrew Luo <an...@outlook.com> on 2023/02/08 16:45:23 UTC

Review Request 74302: RANGER-4076: Support Java 17 for build and runtime

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74302/
-----------------------------------------------------------

Review request for ranger.


Bugs: RANGER-4076
    https://issues.apache.org/jira/browse/RANGER-4076


Repository: ranger


Description
-------

This requires upgrading a few maven plugins, libraries, as well as changing from Nashorn to GraalJS.  This uses the latest version of GraalJS that supports JDK 8.


Diffs
-----

  .mvn/jvm.config PRE-CREATION 
  agents-common/pom.xml b753c1368 
  agents-common/src/main/java/org/apache/ranger/plugin/util/NashornScriptEngineCreator.java db620df92 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ScriptEngineUtil.java 8d76c1d81 
  hdfs-agent/pom.xml a23cdfea0 
  kms/pom.xml e97b993d7 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java 7188b19b2 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java b6fc32950 
  plugin-nestedstructure/pom.xml 0e208f99c 
  plugin-nestedstructure/src/main/java/org/apache/ranger/authorization/nestedstructure/authorizer/RecordFilterJavaScript.java 77767767c 
  plugin-nestedstructure/src/test/java/org/apache/ranger/authorization/nestedstructure/authorizer/TestRecordFilterJavaScript.java 9cb161b8d 
  pom.xml e402bcc5d 
  ranger-trino-plugin-shim/src/main/java/org/apache/ranger/authorization/trino/authorizer/TrinoRangerPlugin.java aa6ce8620 
  security-admin/pom.xml 2e5e21950 


Diff: https://reviews.apache.org/r/74302/diff/1/


Testing
-------

Tested compile and run on both Java 8 and 17.


Thanks,

Andrew Luo


Re: Review Request 74302: RANGER-4076: Support Java 17 for build and runtime

Posted by Andrew Luo <an...@outlook.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74302/
-----------------------------------------------------------

(Updated Feb. 8, 2023, 8:46 p.m.)


Review request for ranger.


Bugs: RANGER-4076
    https://issues.apache.org/jira/browse/RANGER-4076


Repository: ranger


Description
-------

This requires upgrading a few maven plugins, libraries, as well as changing from Nashorn to GraalJS.  This uses the latest version of GraalJS that supports JDK 8.

If preferred, I can split this into smaller commits.


Diffs (updated)
-----

  .mvn/jvm.config PRE-CREATION 
  agents-common/pom.xml b753c1368 
  agents-common/src/main/java/org/apache/ranger/plugin/util/NashornScriptEngineCreator.java db620df92 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ScriptEngineUtil.java 8d76c1d81 
  hdfs-agent/pom.xml a23cdfea0 
  kms/pom.xml e97b993d7 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java 7188b19b2 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java b6fc32950 
  plugin-nestedstructure/pom.xml 0e208f99c 
  plugin-nestedstructure/src/main/java/org/apache/ranger/authorization/nestedstructure/authorizer/RecordFilterJavaScript.java 77767767c 
  plugin-nestedstructure/src/test/java/org/apache/ranger/authorization/nestedstructure/authorizer/TestRecordFilterJavaScript.java 9cb161b8d 
  pom.xml e402bcc5d 
  security-admin/pom.xml 2e5e21950 


Diff: https://reviews.apache.org/r/74302/diff/4/

Changes: https://reviews.apache.org/r/74302/diff/3-4/


Testing
-------

Tested compile and run on both Java 8 and 17.


File Attachments
----------------

0001-RANGER-4076-Support-Java-17-for-build-and-runtime.patch
  https://reviews.apache.org/media/uploaded/files/2023/02/08/d87b4c7c-e84c-4ddb-b059-51c98bf75772__0001-RANGER-4076-Support-Java-17-for-build-and-runtime.patch


Thanks,

Andrew Luo


Re: Review Request 74302: RANGER-4076: Support Java 17 for build and runtime

Posted by Andrew Luo <an...@outlook.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74302/
-----------------------------------------------------------

(Updated Feb. 8, 2023, 8:43 p.m.)


Review request for ranger.


Bugs: RANGER-4076
    https://issues.apache.org/jira/browse/RANGER-4076


Repository: ranger


Description
-------

This requires upgrading a few maven plugins, libraries, as well as changing from Nashorn to GraalJS.  This uses the latest version of GraalJS that supports JDK 8.

If preferred, I can split this into smaller commits.


Diffs (updated)
-----

  .mvn/jvm.config PRE-CREATION 
  agents-common/pom.xml b753c1368 
  agents-common/src/main/java/org/apache/ranger/plugin/util/NashornScriptEngineCreator.java db620df92 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ScriptEngineUtil.java 8d76c1d81 
  hdfs-agent/pom.xml a23cdfea0 
  kms/pom.xml e97b993d7 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java 7188b19b2 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java b6fc32950 
  plugin-nestedstructure/pom.xml 0e208f99c 
  plugin-nestedstructure/src/main/java/org/apache/ranger/authorization/nestedstructure/authorizer/RecordFilterJavaScript.java 77767767c 
  plugin-nestedstructure/src/test/java/org/apache/ranger/authorization/nestedstructure/authorizer/TestRecordFilterJavaScript.java 9cb161b8d 
  pom.xml e402bcc5d 
  security-admin/pom.xml 2e5e21950 


Diff: https://reviews.apache.org/r/74302/diff/3/

Changes: https://reviews.apache.org/r/74302/diff/2-3/


Testing
-------

Tested compile and run on both Java 8 and 17.


File Attachments (updated)
----------------

0001-RANGER-4076-Support-Java-17-for-build-and-runtime.patch
  https://reviews.apache.org/media/uploaded/files/2023/02/08/d87b4c7c-e84c-4ddb-b059-51c98bf75772__0001-RANGER-4076-Support-Java-17-for-build-and-runtime.patch


Thanks,

Andrew Luo


Re: Review Request 74302: RANGER-4076: Support Java 17 for build and runtime

Posted by Andrew Luo <an...@outlook.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74302/
-----------------------------------------------------------

(Updated Feb. 8, 2023, 4:49 p.m.)


Review request for ranger.


Bugs: RANGER-4076
    https://issues.apache.org/jira/browse/RANGER-4076


Repository: ranger


Description
-------

This requires upgrading a few maven plugins, libraries, as well as changing from Nashorn to GraalJS.  This uses the latest version of GraalJS that supports JDK 8.

If preferred, I can split this into smaller commits.


Diffs (updated)
-----

  .mvn/jvm.config PRE-CREATION 
  agents-common/pom.xml b753c1368 
  agents-common/src/main/java/org/apache/ranger/plugin/util/NashornScriptEngineCreator.java db620df92 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ScriptEngineUtil.java 8d76c1d81 
  hdfs-agent/pom.xml a23cdfea0 
  kms/pom.xml e97b993d7 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java 7188b19b2 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java b6fc32950 
  plugin-nestedstructure/pom.xml 0e208f99c 
  plugin-nestedstructure/src/main/java/org/apache/ranger/authorization/nestedstructure/authorizer/RecordFilterJavaScript.java 77767767c 
  plugin-nestedstructure/src/test/java/org/apache/ranger/authorization/nestedstructure/authorizer/TestRecordFilterJavaScript.java 9cb161b8d 
  pom.xml e402bcc5d 
  security-admin/pom.xml 2e5e21950 


Diff: https://reviews.apache.org/r/74302/diff/2/

Changes: https://reviews.apache.org/r/74302/diff/1-2/


Testing
-------

Tested compile and run on both Java 8 and 17.


Thanks,

Andrew Luo


Re: Review Request 74302: RANGER-4076: Support Java 17 for build and runtime

Posted by Andrew Luo <an...@outlook.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74302/
-----------------------------------------------------------

(Updated Feb. 8, 2023, 4:46 p.m.)


Review request for ranger.


Bugs: RANGER-4076
    https://issues.apache.org/jira/browse/RANGER-4076


Repository: ranger


Description (updated)
-------

This requires upgrading a few maven plugins, libraries, as well as changing from Nashorn to GraalJS.  This uses the latest version of GraalJS that supports JDK 8.

If preferred, I can split this into smaller commits.


Diffs
-----

  .mvn/jvm.config PRE-CREATION 
  agents-common/pom.xml b753c1368 
  agents-common/src/main/java/org/apache/ranger/plugin/util/NashornScriptEngineCreator.java db620df92 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ScriptEngineUtil.java 8d76c1d81 
  hdfs-agent/pom.xml a23cdfea0 
  kms/pom.xml e97b993d7 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java 7188b19b2 
  kms/src/main/java/org/apache/hadoop/crypto/key/RangerMasterKey.java b6fc32950 
  plugin-nestedstructure/pom.xml 0e208f99c 
  plugin-nestedstructure/src/main/java/org/apache/ranger/authorization/nestedstructure/authorizer/RecordFilterJavaScript.java 77767767c 
  plugin-nestedstructure/src/test/java/org/apache/ranger/authorization/nestedstructure/authorizer/TestRecordFilterJavaScript.java 9cb161b8d 
  pom.xml e402bcc5d 
  ranger-trino-plugin-shim/src/main/java/org/apache/ranger/authorization/trino/authorizer/TrinoRangerPlugin.java aa6ce8620 
  security-admin/pom.xml 2e5e21950 


Diff: https://reviews.apache.org/r/74302/diff/1/


Testing
-------

Tested compile and run on both Java 8 and 17.


Thanks,

Andrew Luo