Re: Review Request 49735: AMBARI-17570 Lack of importing ClientComponentHasNoStatus
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49735/#review142126 --- Ship it! Ship It! - Sid Wagle On July 6, 2016, 11:45 p.m., Masahiro Tanaka wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49735/ > --- > > (Updated July 6, 2016, 11:45 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Juanjo > Marron. > > > Bugs: AMBARI-17570 > https://issues.apache.org/jira/browse/AMBARI-17570 > > > Repository: ambari > > > Description > --- > > Some of the *_client.py files (e.g. > ambari/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon_client.py) > don't import ClientComponentHasNoStatus while it is used. It should be > imported. > > > Diffs > - > > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon_client.py > 1e7ed1f3 > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_client.py > f8c33dd > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_client.py > 93d244d > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_client.py > 1f85fc0 > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/package/scripts/kerberos_client.py > 5a398db > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_client.py > 7c5b6e0 > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/pig_client.py > 111f4d2 > > ambari-server/src/main/resources/common-services/SLIDER/0.60.0.2.2/package/scripts/slider_client.py > 99314ec > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/package/scripts/zookeeper_client.py > fac92e1 > > Diff: https://reviews.apache.org/r/49735/diff/ > > > Testing > --- > > -1 overall. Here are the results of testing the latest attachment > http://issues.apache.org/jira/secure/attachment/12816319/AMBARI-17570.patch > against trunk revision . > +1 @author. The patch does not contain any @author tags. > -1 tests included. The patch doesn't appear to include any new or modified > tests. > Please justify why no new tests are needed for this patch. > Also please list what manual steps were performed to verify this patch. > +1 javac. The applied patch does not increase the total number of javac > compiler warnings. > +1 release audit. The applied patch does not increase the total number of > release audit warnings. > +1 core tests. The patch passed unit tests in ambari-server. > Test results: > https://builds.apache.org/job/Ambari-trunk-test-patch/7710//testReport/ > Console output: > https://builds.apache.org/job/Ambari-trunk-test-patch/7710//console > This message is automatically generated. > > > Thanks, > > Masahiro Tanaka > >
Re: Review Request 49735: AMBARI-17570 Lack of importing ClientComponentHasNoStatus
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49735/#review141106 --- Ship it! Ship It! - Juanjo Marron On July 6, 2016, 11:45 p.m., Masahiro Tanaka wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49735/ > --- > > (Updated July 6, 2016, 11:45 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Juanjo > Marron. > > > Bugs: AMBARI-17570 > https://issues.apache.org/jira/browse/AMBARI-17570 > > > Repository: ambari > > > Description > --- > > Some of the *_client.py files (e.g. > ambari/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon_client.py) > don't import ClientComponentHasNoStatus while it is used. It should be > imported. > > > Diffs > - > > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon_client.py > 1e7ed1f3 > > ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_client.py > f8c33dd > > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_client.py > 93d244d > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_client.py > 1f85fc0 > > ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/package/scripts/kerberos_client.py > 5a398db > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_client.py > 7c5b6e0 > > ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/pig_client.py > 111f4d2 > > ambari-server/src/main/resources/common-services/SLIDER/0.60.0.2.2/package/scripts/slider_client.py > 99314ec > > ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/package/scripts/zookeeper_client.py > fac92e1 > > Diff: https://reviews.apache.org/r/49735/diff/ > > > Testing > --- > > -1 overall. Here are the results of testing the latest attachment > http://issues.apache.org/jira/secure/attachment/12816319/AMBARI-17570.patch > against trunk revision . > +1 @author. The patch does not contain any @author tags. > -1 tests included. The patch doesn't appear to include any new or modified > tests. > Please justify why no new tests are needed for this patch. > Also please list what manual steps were performed to verify this patch. > +1 javac. The applied patch does not increase the total number of javac > compiler warnings. > +1 release audit. The applied patch does not increase the total number of > release audit warnings. > +1 core tests. The patch passed unit tests in ambari-server. > Test results: > https://builds.apache.org/job/Ambari-trunk-test-patch/7710//testReport/ > Console output: > https://builds.apache.org/job/Ambari-trunk-test-patch/7710//console > This message is automatically generated. > > > Thanks, > > Masahiro Tanaka > >
Review Request 49735: AMBARI-17570 Lack of importing ClientComponentHasNoStatus
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49735/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Juanjo Marron. Bugs: AMBARI-17570 https://issues.apache.org/jira/browse/AMBARI-17570 Repository: ambari Description --- Some of the *_client.py files (e.g. ambari/ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon_client.py) don't import ClientComponentHasNoStatus while it is used. It should be imported. Diffs - ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon_client.py 1e7ed1f3 ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase_client.py f8c33dd ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/hdfs_client.py 93d244d ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_client.py 1f85fc0 ambari-server/src/main/resources/common-services/KERBEROS/1.10.3-10/package/scripts/kerberos_client.py 5a398db ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_client.py 7c5b6e0 ambari-server/src/main/resources/common-services/PIG/0.12.0.2.0/package/scripts/pig_client.py 111f4d2 ambari-server/src/main/resources/common-services/SLIDER/0.60.0.2.2/package/scripts/slider_client.py 99314ec ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/package/scripts/zookeeper_client.py fac92e1 Diff: https://reviews.apache.org/r/49735/diff/ Testing --- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12816319/AMBARI-17570.patch against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed unit tests in ambari-server. Test results: https://builds.apache.org/job/Ambari-trunk-test-patch/7710//testReport/ Console output: https://builds.apache.org/job/Ambari-trunk-test-patch/7710//console This message is automatically generated. Thanks, Masahiro Tanaka