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

(Updated Oct. 25, 2023, 10:10 a.m.)


Review request for ranger, Abhishek  Kumar, bhavik patel, Dhaval Shah, 
Dineshkumar Yadav, Kishor Gollapalliwar, Abhay Kulkarni, Madhan Neethiraj, 
Mehul Parikh, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy.


Changes
-------

Added fix for ranger-kms and addressed the review comment.


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


Repository: ranger


Description
-------

**Problem Statement:** Currently db_setup.py file does not accepts or add 
JAVA_OPTS options in java commands executed from db_setup.py and 
changepasswordutil.py file. In certain environment user may need these options 
to execute the java commands successfully.

**Proposed Solution:** 
In the proposed patch, db_setup.py and changepasswordutil.py file can read 
JAVA_OPTS from environment variable and pass that to java class run command 
being executed from the same file.
Alternatively user can provide the same in install.propeties via java_opts 
property. setup.sh shall export the java_opts value as JAVA_OPTS environement 
variable.


Diffs (updated)
-----

  kms/scripts/db_setup.py d21a6ac75 
  kms/scripts/install.properties 0e5da3c75 
  kms/scripts/setup.sh f723e09bb 
  security-admin/scripts/changepasswordutil.py e45dab643 
  security-admin/scripts/db_setup.py 24502f4fb 
  security-admin/scripts/install.properties b8e864e9a 
  security-admin/scripts/setup.sh 750455f08 


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

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


Testing
-------

Steps:
1. Exported a JAVA_OPTS option '-Xms1g' by running the command export 
JAVA_OPTS='-Xms1g' on the console.
2. Added debug statement in db_setup.py file to print java class run command 
which are usually executed when db_setup.py is called.
2. Executed the setup.sh file
3. setup.sh command executed succesfully and can see debug logs with added 
JAVA_OPTS '-Xms1g'


Thanks,

Pradeep Agrawal

Reply via email to