----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4998/#review7523 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/service/AuthorizationService.java <https://reviews.apache.org/r/4998/#comment16655> It was identified in oozie-797 that conf.getStrings() may fail to parse values if there are whitespaces. Can you check oozie-797 and see whether similar issue can happen here or not? trunk/docs/src/site/twiki/AG_Install.twiki <https://reviews.apache.org/r/4998/#comment16656> do we need to specify its comma-seperated list? - Virag On 2012-05-03 20:28:59, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4998/ > ----------------------------------------------------------- > > (Updated 2012-05-03 20:28:59) > > > Review request for oozie. > > > Summary > ------- > > adds the option to specify a list of admin groups to check users for admin > privileges instead of using the adminusers.txt file. the adminusers.txt file > option is still available if the admin groups list is not specified. > > > This addresses bug OOZIE-773. > https://issues.apache.org/jira/browse/OOZIE-773 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/service/AuthorizationService.java > 1333606 > > trunk/core/src/test/java/org/apache/oozie/service/TestAuthorizationService.java > 1333606 > trunk/docs/src/site/twiki/AG_Install.twiki 1333606 > > Diff: https://reviews.apache.org/r/4998/diff > > > Testing > ------- > > run all tests successfully > > > Thanks, > > Alejandro > >
