Re: Review Request 69083: RANGER-2251 : Need to provide options for making java heap size memory configurable in Ranger services.

2018-10-22 Thread Pradeep Agrawal 

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


Ship it!




- Pradeep Agrawal


On Oct. 22, 2018, 9:01 a.m., Vishal Suvagia wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69083/
> ---
> 
> (Updated Oct. 22, 2018, 9:01 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Colm O hEigeartaigh, Gautam Borad, 
> Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
> Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2251
> https://issues.apache.org/jira/browse/RANGER-2251
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Need to make java heap size memory configurable for Ranger services.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 
> 990d3c7922351f298277792baa2551efa5e7a1cc 
>   kms/scripts/ranger-kms 604d7014c5584d5feef26975c7bfffd8c2194f1e 
>   security-admin/scripts/db_setup.py 5ac312fba9c3ddfb8c345d2c2551bab9c49fd67b 
>   security-admin/scripts/install.properties 
> 34c52ebe58b59892ebf5f8fd66d81a73264aa049 
>   tagsync/scripts/ranger-tagsync-services.sh 
> 6fcdf1562569f6203da309936e4762395c9036f0 
>   unixauthservice/scripts/ranger-usersync-services.sh 
> 0c03c5a18eb9a15740df8398e96fc14104277dd2 
> 
> 
> Diff: https://reviews.apache.org/r/69083/diff/2/
> 
> 
> Testing
> ---
> 
> Tested with fresh installation for heapsize to be effective for Ranger: 
> Admin, Usersycnc, Tagsync and KMS services.
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>

Re: Review Request 69083: RANGER-2251 : Need to provide options for making java heap size memory configurable in Ranger services.

2018-10-22 Thread Abhay Kulkarni 

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


Ship it!




Ship It!

- Abhay Kulkarni


On Oct. 22, 2018, 9:01 a.m., Vishal Suvagia wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69083/
> ---
> 
> (Updated Oct. 22, 2018, 9:01 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Colm O hEigeartaigh, Gautam Borad, 
> Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
> Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2251
> https://issues.apache.org/jira/browse/RANGER-2251
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Need to make java heap size memory configurable for Ranger services.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 
> 990d3c7922351f298277792baa2551efa5e7a1cc 
>   kms/scripts/ranger-kms 604d7014c5584d5feef26975c7bfffd8c2194f1e 
>   security-admin/scripts/db_setup.py 5ac312fba9c3ddfb8c345d2c2551bab9c49fd67b 
>   security-admin/scripts/install.properties 
> 34c52ebe58b59892ebf5f8fd66d81a73264aa049 
>   tagsync/scripts/ranger-tagsync-services.sh 
> 6fcdf1562569f6203da309936e4762395c9036f0 
>   unixauthservice/scripts/ranger-usersync-services.sh 
> 0c03c5a18eb9a15740df8398e96fc14104277dd2 
> 
> 
> Diff: https://reviews.apache.org/r/69083/diff/2/
> 
> 
> Testing
> ---
> 
> Tested with fresh installation for heapsize to be effective for Ranger: 
> Admin, Usersycnc, Tagsync and KMS services.
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>

Re: Review Request 69083: RANGER-2251 : Need to provide options for making java heap size memory configurable in Ranger services.

2018-10-22 Thread Velmurugan Periasamy 

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


Ship it!




Ship It!

- Velmurugan Periasamy


On Oct. 22, 2018, 9:01 a.m., Vishal Suvagia wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69083/
> ---
> 
> (Updated Oct. 22, 2018, 9:01 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Colm O hEigeartaigh, Gautam Borad, 
> Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
> Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2251
> https://issues.apache.org/jira/browse/RANGER-2251
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Need to make java heap size memory configurable for Ranger services.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 
> 990d3c7922351f298277792baa2551efa5e7a1cc 
>   kms/scripts/ranger-kms 604d7014c5584d5feef26975c7bfffd8c2194f1e 
>   security-admin/scripts/db_setup.py 5ac312fba9c3ddfb8c345d2c2551bab9c49fd67b 
>   security-admin/scripts/install.properties 
> 34c52ebe58b59892ebf5f8fd66d81a73264aa049 
>   tagsync/scripts/ranger-tagsync-services.sh 
> 6fcdf1562569f6203da309936e4762395c9036f0 
>   unixauthservice/scripts/ranger-usersync-services.sh 
> 0c03c5a18eb9a15740df8398e96fc14104277dd2 
> 
> 
> Diff: https://reviews.apache.org/r/69083/diff/2/
> 
> 
> Testing
> ---
> 
> Tested with fresh installation for heapsize to be effective for Ranger: 
> Admin, Usersycnc, Tagsync and KMS services.
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>

Re: Review Request 69083: RANGER-2251 : Need to provide options for making java heap size memory configurable in Ranger services.

2018-10-22 Thread Vishal Suvagia via Review Board 

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

(Updated Oct. 22, 2018, 9:01 a.m.)


Review request for ranger, Ankita Sinha, Colm O hEigeartaigh, Gautam Borad, 
Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
Sailaja Polavarapu, and Velmurugan Periasamy.


Changes
---

Updating review request.


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


Repository: ranger


Description
---

Need to make java heap size memory configurable for Ranger services.


Diffs (updated)
-

  embeddedwebserver/scripts/ranger-admin-services.sh 
990d3c7922351f298277792baa2551efa5e7a1cc 
  kms/scripts/ranger-kms 604d7014c5584d5feef26975c7bfffd8c2194f1e 
  security-admin/scripts/db_setup.py 5ac312fba9c3ddfb8c345d2c2551bab9c49fd67b 
  security-admin/scripts/install.properties 
34c52ebe58b59892ebf5f8fd66d81a73264aa049 
  tagsync/scripts/ranger-tagsync-services.sh 
6fcdf1562569f6203da309936e4762395c9036f0 
  unixauthservice/scripts/ranger-usersync-services.sh 
0c03c5a18eb9a15740df8398e96fc14104277dd2 


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

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


Testing
---

Tested with fresh installation for heapsize to be effective for Ranger: Admin, 
Usersycnc, Tagsync and KMS services.


Thanks,

Vishal Suvagia

Re: Review Request 69083: RANGER-2251 : Need to provide options for making java heap size memory configurable in Ranger services.

2018-10-19 Thread Abhay Kulkarni 

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




unixauthservice/scripts/setup.py
Line 554 (original), 554 (patched)


Why is this change included in this patch? Why is this necessary?



unixauthservice/scripts/setup.py
Line 561 (original), 561 (patched)


Why is this change necessary? Why is it included in this patch?


- Abhay Kulkarni


On Oct. 19, 2018, 1:37 p.m., Vishal Suvagia wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69083/
> ---
> 
> (Updated Oct. 19, 2018, 1:37 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Colm O hEigeartaigh, Gautam Borad, 
> Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
> Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2251
> https://issues.apache.org/jira/browse/RANGER-2251
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Need to make java heap size memory configurable for Ranger services.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 
> 990d3c7922351f298277792baa2551efa5e7a1cc 
>   kms/scripts/ranger-kms 604d7014c5584d5feef26975c7bfffd8c2194f1e 
>   security-admin/scripts/db_setup.py 5ac312fba9c3ddfb8c345d2c2551bab9c49fd67b 
>   security-admin/scripts/install.properties 
> 34c52ebe58b59892ebf5f8fd66d81a73264aa049 
>   tagsync/scripts/ranger-tagsync-services.sh 
> 6fcdf1562569f6203da309936e4762395c9036f0 
>   unixauthservice/scripts/ranger-usersync-services.sh 
> 0c03c5a18eb9a15740df8398e96fc14104277dd2 
>   unixauthservice/scripts/setup.py e0c8c830ff13aa8abae3c0b20e89e2a27a07d099 
> 
> 
> Diff: https://reviews.apache.org/r/69083/diff/1/
> 
> 
> Testing
> ---
> 
> Tested with fresh installation for heapsize to be effective for Ranger: 
> Admin, Usersycnc, Tagsync and KMS services.
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>

Review Request 69083: RANGER-2251 : Need to provide options for making java heap size memory configurable in Ranger services.

2018-10-19 Thread Vishal Suvagia via Review Board 

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

Review request for ranger, Ankita Sinha, Colm O hEigeartaigh, Gautam Borad, 
Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
Sailaja Polavarapu, and Velmurugan Periasamy.


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


Repository: ranger


Description
---

Need to make java heap size memory configurable for Ranger services.


Diffs
-

  embeddedwebserver/scripts/ranger-admin-services.sh 
990d3c7922351f298277792baa2551efa5e7a1cc 
  kms/scripts/ranger-kms 604d7014c5584d5feef26975c7bfffd8c2194f1e 
  security-admin/scripts/db_setup.py 5ac312fba9c3ddfb8c345d2c2551bab9c49fd67b 
  security-admin/scripts/install.properties 
34c52ebe58b59892ebf5f8fd66d81a73264aa049 
  tagsync/scripts/ranger-tagsync-services.sh 
6fcdf1562569f6203da309936e4762395c9036f0 
  unixauthservice/scripts/ranger-usersync-services.sh 
0c03c5a18eb9a15740df8398e96fc14104277dd2 
  unixauthservice/scripts/setup.py e0c8c830ff13aa8abae3c0b20e89e2a27a07d099 


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


Testing
---

Tested with fresh installation for heapsize to be effective for Ranger: Admin, 
Usersycnc, Tagsync and KMS services.


Thanks,

Vishal Suvagia