[jira] [Resolved] (RANGER-1723) The Ranger doesn't support solr6. In the HttpSolrClient class of solr6, the setMaxRetries function has been removed, which causes ranger to fail to compile.

2017-08-08 Thread peng.jianhua (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] peng.jianhua resolved RANGER-1723. -- Resolution: Fixed Fix Version/s: 1.0.0 > The Ranger doesn't support solr6. In the

Re: Review Request 60732: RANGER-1491 : Automatically map group of external users to Administrator Role

2017-08-08 Thread bhavik patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60732/ --- (Updated Aug. 8, 2017, 8:31 a.m.) Review request for ranger, Ankita Sinha, Don

[jira] [Updated] (RANGER-1491) Automatically map group of external users to Administrator Role

2017-08-08 Thread bhavik patel (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1491?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] bhavik patel updated RANGER-1491: - Attachment: RANGER-1491-07-03.patch > Automatically map group of external users to Administrator

Re: Review Request 61202: New Defects reported by Coverity Scan for Apache Ranger

2017-08-08 Thread pengjianhua
> On 八月 8, 2017, 3:56 p.m., Colm O hEigeartaigh wrote: > > Thanks, indentation looks fine now. I'd like to see just a few minor > > improvements to the code in RangerHiveAuthorizer. > > > > 1) The "if" statement could be moved up to the top under the for loop to > > avoid creating unnecessary

Re: Review Request 61202: New Defects reported by Coverity Scan for Apache Ranger

2017-08-08 Thread pengjianhua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61202/ --- (Updated 八月 9, 2017, 5:35 a.m.) Review request for ranger, Don Bosco Durai,

Re: Review Request 61452: RANGER-1725: It is incorrect to judge whether you need to create a RangerServiceDef object for a given service

2017-08-08 Thread pengjianhua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61452/ --- (Updated 八月 9, 2017, 5:33 a.m.) Review request for ranger, Alok Lal, Ankita

[jira] [Created] (RANGER-1727) Ranger allows user to change an external user's password with 'null' old password

2017-08-08 Thread Pradeep Agrawal (JIRA)
Pradeep Agrawal created RANGER-1727: --- Summary: Ranger allows user to change an external user's password with 'null' old password Key: RANGER-1727 URL: https://issues.apache.org/jira/browse/RANGER-1727

[jira] [Updated] (RANGER-1725) It is incorrect to judge whether you need to create a RangerServiceDef object for a given service

2017-08-08 Thread peng.jianhua (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1725?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] peng.jianhua updated RANGER-1725: - Attachment: (was: 0001-RANGER-1725-It-is-incorrect-to-judge-whether-you-nee.patch) > It is

[jira] [Updated] (RANGER-1725) It is incorrect to judge whether you need to create a RangerServiceDef object for a given service

2017-08-08 Thread peng.jianhua (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1725?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] peng.jianhua updated RANGER-1725: - Attachment: 0001-RANGER-1725-It-is-incorrect-to-judge-whether-you-nee.patch > It is incorrect to

Re: Review Request 61452: RANGER-1725: It is incorrect to judge whether you need to create a RangerServiceDef object for a given service

2017-08-08 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61452/#review182401 --- Looks fine. Could you remove the try/catch statement in

Re: Review Request 61412: RANGER-1649:Ranger Solr Plugin fails to refresh policy due to failure in ticket renewal mechanism

2017-08-08 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61412/#review182403 --- Thanks for the explanation! You could delete the "authWithConfig"

Re: Review Request 61384: RANGER-1722: Duplicate code for exception wrapping in BaseClient

2017-08-08 Thread Zsombor Gegesy
> On Aug. 8, 2017, 4:13 p.m., Colm O hEigeartaigh wrote: > > There's a change in logic I'm a bit unsure about for the "first" case. The > > exception message is now changed from: > > > > msgDesc + errMsg > > > > to > > > > getMessage(exp) + DEFAULT_ERROR_MESSAGE > > > > But the exp is null

Re: Review Request 61384: RANGER-1722: Duplicate code for exception wrapping in BaseClient

2017-08-08 Thread Zsombor Gegesy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61384/ --- (Updated Aug. 8, 2017, 7:37 p.m.) Review request for ranger. Bugs:

Re: Review Request 61202: New Defects reported by Coverity Scan for Apache Ranger

2017-08-08 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61202/#review182398 --- Thanks, indentation looks fine now. I'd like to see just a few

Re: Review Request 61384: RANGER-1722: Duplicate code for exception wrapping in BaseClient

2017-08-08 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61384/#review182399 --- There's a change in logic I'm a bit unsure about for the "first"