----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32683/#review78355 -----------------------------------------------------------
Ship it! Ship It! - Ramesh Mani On March 31, 2015, 6:42 a.m., Madhan Neethiraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32683/ > ----------------------------------------------------------- > > (Updated March 31, 2015, 6:42 a.m.) > > > Review request for ranger, Alok Lal, Don Bosco Durai, dilli dorai, Gautam > Borad, Abhay Kulkarni, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan > Periasamy. > > > Bugs: RANGER-352 > https://issues.apache.org/jira/browse/RANGER-352 > > > Repository: ranger > > > Description > ------- > > RANGER-352: ID attribute added to objects contained by service-def, to > facilitate update. In addition RangerServiceDef.getName()/setName() methods > renamed to RangerServiceDef.getType()/setType() > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicy.java > 3092c79 > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceDef.java > 62be788 > agents-common/src/main/resources/service-defs/ranger-servicedef-hbase.json > e611895 > agents-common/src/main/resources/service-defs/ranger-servicedef-hdfs.json > b279f0c > agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json > aabc73b > agents-common/src/main/resources/service-defs/ranger-servicedef-knox.json > 0174e96 > agents-common/src/main/resources/service-defs/ranger-servicedef-solr.json > 6ae7a87 > agents-common/src/main/resources/service-defs/ranger-servicedef-storm.json > bcc4394 > agents-common/src/main/resources/service-defs/ranger-servicedef-yarn.json > 8af8f9d > > agents-common/src/test/java/org/apache/ranger/plugin/store/TestServiceStore.java > ae04e28 > > security-admin/src/main/java/org/apache/ranger/service/RangerServiceDefServiceBase.java > 48fd7f8 > > Diff: https://reviews.apache.org/r/32683/diff/ > > > Testing > ------- > > Verified that the build and unit test succeed > > > Thanks, > > Madhan Neethiraj > >
