> On March 31, 2016, 6:12 a.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py, > > line 114 > > <https://reviews.apache.org/r/45522/diff/1/?file=1320297#file1320297line114> > > > > Let me check if this restriction for 2.2.1.0 is Oozie specific. > > Juanjo Marron wrote: > Maintained for now in the new patch. Please, let me know if it needs to > be edited > > Jayush Luniya wrote: > @Juanjo. I didnt get a chance to look into this. For now lets leave these > feature as LZO itself and since your logic is correct go ahead with the > change. We can rename if required the feature to oozie_lzo or something like > that if this is Oozie specific. > > Juanjo Marron wrote: > Thanks @Jayush. > I did that and the new patch was uploaded and it is available to be > reviewed and committed > > Jayush Luniya wrote: > Yes, committed to trunk
Thanks! - Juanjo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45522/#review126274 ----------------------------------------------------------- On April 1, 2016, 5:31 p.m., Juanjo Marron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45522/ > ----------------------------------------------------------- > > (Updated April 1, 2016, 5:31 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Jayush Luniya. > > > Bugs: AMBARI-14472 > https://issues.apache.org/jira/browse/AMBARI-14472 > > > Repository: ambari > > > Description > ------- > > Stack Featurize Oozie Service > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/constants.py > 175b889 > > ambari-common/src/main/python/resource_management/libraries/functions/stack_features.py > 6169699 > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/alerts/alert_check_oozie_server.py > 90851c8 > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py > 3b01802 > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_client.py > 4fc50d2 > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server.py > e9da71b > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py > 2db3672 > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py > ce44d5c > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/status_params.py > a08ae3a > > ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json > fd3fbde > > Diff: https://reviews.apache.org/r/45522/diff/ > > > Testing > ------- > > mvn clean test -DskipSurefireTests > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 3:58.802s > [INFO] Finished at: Wed Mar 30 18:57:06 PDT 2016 > [INFO] Final Memory: 110M/796M > [INFO] > ------------------------------------------------------------------------ > [root@localhost ambari]# > > > Thanks, > > Juanjo Marron > >
