[jira] [Created] (RANGER-1612) When servicedef is accessed, one of the properties "enableDenyPolicies" is returned as "false" if there is no value set for it.

2017-05-24 Thread Anuja Leekha (JIRA)
Anuja Leekha created RANGER-1612: Summary: When servicedef is accessed, one of the properties "enableDenyPolicies" is returned as "false" if there is no value set for it. Key: RANGER-1612 URL:

Re: 答复: Re: Patches with trivial changes

2017-05-24 Thread Madhan Neethiraj
Colm, Peng.jianhua, > I will encourage future review requests that have a very trivial spelling > fix to hold on to the fix for a while, so that we can fix multiple spelling > fixes etc. at the same time. Consolidating trivial updates into a single patch/JIRA will certainly help. However,

Re: Patches with trivial changes

2017-05-24 Thread Madhan Neethiraj
Zsombor, >> Instead of discouraging contributions My suggestion was to discourage patches that have *only* very, very tiny and non-essential updates – like changing case of a label word from “G” to “g”, removing a blank line, etc; or patches that touch hundreds of files for

[jira] [Resolved] (RANGER-1575) Some users hope that the pid file of the Ranger Admin can be unified management when they integrate Ranger into the big data platform or business systems to uniform ins

2017-05-24 Thread Qiang Zhang (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1575?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Qiang Zhang resolved RANGER-1575. - Resolution: Fixed > Some users hope that the pid file of the Ranger Admin can be unified >

Re: Review Request 59255: Some users hope that the pid file of the Ranger Admin can be unified management when they integrate Ranger into the big data platform or business systems to uniform install a

2017-05-24 Thread Qiang Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59255/#review175947 --- Ship it! Ship It! - Qiang Zhang On 五月 23, 2017, 4:07 a.m.,

Re: Patches with trivial changes

2017-05-24 Thread Zsombor
Instead of discouraging contributions, the project should ease the contribution - and also the review process. The current JIRA + Reviewboard infrastructure needs a lot's of unnecessary manual steps, which hurts everyone who wants to help. Currently, the contributor needs to do the following,

Review Request 59523: RANGER-689 - "For Solr plugin, use resources folders for adding Ranger properties".

2017-05-24 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59523/ --- Review request for ranger. Bugs: RANGER-689

[jira] [Updated] (RANGER-689) For Solr plugin, use resources folders for adding Ranger properties

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh updated RANGER-689: --- Attachment: 0001-RANGER-689-For-Solr-plugin-use-resources-folders-for.patch > For

[jira] [Updated] (RANGER-689) For Solr plugin, use resources folders for adding Ranger properties

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh updated RANGER-689: --- Fix Version/s: 1.0.0 > For Solr plugin, use resources folders for adding Ranger

[jira] [Commented] (RANGER-689) For Solr plugin, use resources folders for adding Ranger properties

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-689?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16022697#comment-16022697 ] Colm O hEigeartaigh commented on RANGER-689: Putting the jars into service/lib/ext does not

[jira] [Updated] (RANGER-689) For Solr plugin, use resources folders for adding Ranger properties

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh updated RANGER-689: --- Summary: For Solr plugin, use resources folders for adding Ranger properties (was:

[jira] [Assigned] (RANGER-689) For Solr plugin, use lib/ext and resources folders for adding Ranger jars and properties

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-689?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh reassigned RANGER-689: -- Assignee: Colm O hEigeartaigh > For Solr plugin, use lib/ext and resources

Re: Patches with trivial changes

2017-05-24 Thread Gautam Borad
+1 for Madhan's recommendations, Colm, I agree that we should not discourage new contributions. However, I think, we should also not encourage such single line/whitespace contributions. We want contributors who can do more functionals/feature changes and while doing that they can also fix the

答复: Re: Patches with trivial changes

2017-05-24 Thread peng.jianhua
Hi Madhan, Merging these small issues to other branches will increase the workload and the probability of error. If possible, we can undertake similar tasks when some important issues need to be merged into other branches. With the improvement of quality, these small issues will be

[jira] [Updated] (RANGER-1584) Allow tagsync to support log directory from Ambari configuration

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1584?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh updated RANGER-1584: Fix Version/s: 1.0.0 > Allow tagsync to support log directory from Ambari

Re: Review Request 59255: Some users hope that the pid file of the Ranger Admin can be unified management when they integrate Ranger into the big data platform or business systems to uniform install a

2017-05-24 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59255/#review175914 --- Ship it! Ship It! - Colm O hEigeartaigh On May 23, 2017,

[jira] [Closed] (RANGER-1049) Testing connection at a Kerberos server

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1049?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh closed RANGER-1049. --- > Testing connection at a Kerberos server > --- > >

[jira] [Closed] (RANGER-1050) Change my name on all systems

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh closed RANGER-1050. --- > Change my name on all systems > - > > Key:

[jira] [Resolved] (RANGER-1050) Change my name on all systems

2017-05-24 Thread Colm O hEigeartaigh (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colm O hEigeartaigh resolved RANGER-1050. - Resolution: Not A Problem > Change my name on all systems >

Re: Patches with trivial changes

2017-05-24 Thread Colm O hEigeartaigh
Hi Madhan, Trivial commits provide a path to get new contributors on board to the project - something that the project needs IMO. Yes it may make backporting fixes a little more difficult, but it's hardly an intractable problem to figure out some whitespace changes between branches - it's not as

[jira] [Resolved] (RANGER-1610) Modify the Logger scope and keep the consistent info in XPolicyService.java

2017-05-24 Thread Qiang Zhang (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1610?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Qiang Zhang resolved RANGER-1610. - Resolution: Invalid > Modify the Logger scope and keep the consistent info in

[jira] [Assigned] (RANGER-1610) Modify the Logger scope and keep the consistent info in XPolicyService.java

2017-05-24 Thread Qiang Zhang (JIRA)
[ https://issues.apache.org/jira/browse/RANGER-1610?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Qiang Zhang reassigned RANGER-1610: --- Assignee: Qiang Zhang > Modify the Logger scope and keep the consistent info in

Patches with trivial changes

2017-05-24 Thread Madhan Neethiraj
All, I notice a number of recent patches address trivial issues like white space, spelling mistakes (one patch just changed a single letter in a label). And few other patches update a large number of files for trivial/non-functional changes – like whitespaces. I strongly suggest we refrain