Re: Review Request 73505: RANGER-3355 : Update the current logging mechanism to use custom log4j conf.

2021-08-09 Thread Jayendra Parab

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


Ship it!




Ship It!

- Jayendra Parab


On Aug. 6, 2021, 2:38 p.m., Mateen Mansoori wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73505/
> ---
> 
> (Updated Aug. 6, 2021, 2:38 p.m.)
> 
> 
> Review request for ranger, Dineshkumar Yadav, Jayendra Parab, Abhay Kulkarni, 
> Mehul Parikh, Mugdha Varadkar, Pradeep Agrawal, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3355
> https://issues.apache.org/jira/browse/RANGER-3355
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Update the current logging mechanism to use custom log4j.properties file.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 0bc06e1d1 
>   security-admin/scripts/db_setup.py 4dcf6c98f 
>   security-admin/scripts/install.properties 6cde15dff 
>   security-admin/scripts/setup.sh df8d64f0c 
>   security-admin/src/main/webapp/WEB-INF/log4j.properties b47554cdc 
> 
> 
> Diff: https://reviews.apache.org/r/73505/diff/1/
> 
> 
> Testing
> ---
> 
> Tested on local VM.
> Tested : By providing custom location for log4j conf file, Also with defualt 
> log4j conf and default log directory.
> 
> 
> Thanks,
> 
> Mateen Mansoori
> 
>



Re: Review Request 73505: RANGER-3355 : Update the current logging mechanism to use custom log4j conf.

2021-08-09 Thread Mehul Parikh

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


Ship it!




Ship It!

- Mehul Parikh


On Aug. 6, 2021, 2:38 p.m., Mateen Mansoori wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73505/
> ---
> 
> (Updated Aug. 6, 2021, 2:38 p.m.)
> 
> 
> Review request for ranger, Dineshkumar Yadav, Jayendra Parab, Abhay Kulkarni, 
> Mehul Parikh, Mugdha Varadkar, Pradeep Agrawal, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3355
> https://issues.apache.org/jira/browse/RANGER-3355
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Update the current logging mechanism to use custom log4j.properties file.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 0bc06e1d1 
>   security-admin/scripts/db_setup.py 4dcf6c98f 
>   security-admin/scripts/install.properties 6cde15dff 
>   security-admin/scripts/setup.sh df8d64f0c 
>   security-admin/src/main/webapp/WEB-INF/log4j.properties b47554cdc 
> 
> 
> Diff: https://reviews.apache.org/r/73505/diff/1/
> 
> 
> Testing
> ---
> 
> Tested on local VM.
> Tested : By providing custom location for log4j conf file, Also with defualt 
> log4j conf and default log directory.
> 
> 
> Thanks,
> 
> Mateen Mansoori
> 
>



Re: Review Request 73505: RANGER-3355 : Update the current logging mechanism to use custom log4j conf.

2021-08-09 Thread Vishal Suvagia via Review Board

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


Ship it!




Ship It!

- Vishal Suvagia


On Aug. 6, 2021, 2:38 p.m., Mateen Mansoori wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73505/
> ---
> 
> (Updated Aug. 6, 2021, 2:38 p.m.)
> 
> 
> Review request for ranger, Dineshkumar Yadav, Jayendra Parab, Abhay Kulkarni, 
> Mehul Parikh, Mugdha Varadkar, Pradeep Agrawal, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3355
> https://issues.apache.org/jira/browse/RANGER-3355
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> Update the current logging mechanism to use custom log4j.properties file.
> 
> 
> Diffs
> -
> 
>   embeddedwebserver/scripts/ranger-admin-services.sh 0bc06e1d1 
>   security-admin/scripts/db_setup.py 4dcf6c98f 
>   security-admin/scripts/install.properties 6cde15dff 
>   security-admin/scripts/setup.sh df8d64f0c 
>   security-admin/src/main/webapp/WEB-INF/log4j.properties b47554cdc 
> 
> 
> Diff: https://reviews.apache.org/r/73505/diff/1/
> 
> 
> Testing
> ---
> 
> Tested on local VM.
> Tested : By providing custom location for log4j conf file, Also with defualt 
> log4j conf and default log directory.
> 
> 
> Thanks,
> 
> Mateen Mansoori
> 
>



Re: Review Request 73505: RANGER-3355 : Update the current logging mechanism to use custom log4j conf.

2021-08-06 Thread Mateen Mansoori

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

(Updated Aug. 6, 2021, 2:38 p.m.)


Review request for ranger, Dineshkumar Yadav, Jayendra Parab, Abhay Kulkarni, 
Mehul Parikh, Mugdha Varadkar, Pradeep Agrawal, and Velmurugan Periasamy.


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


Repository: ranger


Description
---

Update the current logging mechanism to use custom log4j.properties file.


Diffs
-

  embeddedwebserver/scripts/ranger-admin-services.sh 0bc06e1d1 
  security-admin/scripts/db_setup.py 4dcf6c98f 
  security-admin/scripts/install.properties 6cde15dff 
  security-admin/scripts/setup.sh df8d64f0c 
  security-admin/src/main/webapp/WEB-INF/log4j.properties b47554cdc 


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


Testing (updated)
---

Tested on local VM.
Tested : By providing custom location for log4j conf file, Also with defualt 
log4j conf and default log directory.


Thanks,

Mateen Mansoori



Review Request 73505: RANGER-3355 : Update the current logging mechanism to use custom log4j conf.

2021-08-05 Thread Mateen Mansoori

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

Review request for ranger, Dineshkumar Yadav, Abhay Kulkarni, Mehul Parikh, 
Mugdha Varadkar, Pradeep Agrawal, and Velmurugan Periasamy.


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


Repository: ranger


Description
---

Update the current logging mechanism to use custom log4j.properties file.


Diffs
-

  embeddedwebserver/scripts/ranger-admin-services.sh 0bc06e1d1 
  security-admin/scripts/db_setup.py 4dcf6c98f 
  security-admin/scripts/install.properties 6cde15dff 
  security-admin/scripts/setup.sh df8d64f0c 
  security-admin/src/main/webapp/WEB-INF/log4j.properties b47554cdc 


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


Testing
---

Tested on local VM.


Thanks,

Mateen Mansoori