Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15687 )

Change subject: IMPALA-9618: fix some usability issues with dev env
......................................................................


Patch Set 7:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/15687/7/bin/impala-config.sh
File bin/impala-config.sh:

http://gerrit.cloudera.org:8080/#/c/15687/7/bin/impala-config.sh@369
PS7, Line 369: export 
IMPALA_CLUSTER_NODES_DIR="${IMPALA_CLUSTER_NODES_DIR-$IMPALA_HOME/testdata/cluster/cdh$CDH_MAJOR_VERSION}"
line too long (113 > 90)


http://gerrit.cloudera.org:8080/#/c/15687/7/tests/comparison/cluster.py
File tests/comparison/cluster.py:

http://gerrit.cloudera.org:8080/#/c/15687/7/tests/comparison/cluster.py@212
PS7, Line 212:
flake8: E501 line too long (98 > 90 characters)



--
To view, visit http://gerrit.cloudera.org:8080/15687
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd8b42a6d045d73e3ea29015aa6ccbbde278eec7
Gerrit-Change-Number: 15687
Gerrit-PatchSet: 7
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Thu, 09 Apr 2020 03:05:40 +0000
Gerrit-HasComments: Yes

Reply via email to