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

Change subject: IMPALA-8950: Add -d, -f options to hdfs copyFromLocal, put, cp
......................................................................


Patch Set 3:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/14311/3/tests/query_test/test_hdfs_file_mods.py
File tests/query_test/test_hdfs_file_mods.py:

http://gerrit.cloudera.org:8080/#/c/14311/3/tests/query_test/test_hdfs_file_mods.py@59
PS3, Line 59: r
flake8: E501 line too long (94 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/14311/3/tests/query_test/test_multiple_filesystems.py
File tests/query_test/test_multiple_filesystems.py:

http://gerrit.cloudera.org:8080/#/c/14311/3/tests/query_test/test_multiple_filesystems.py@52
PS3, Line 52: (
flake8: E501 line too long (91 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/14311/3/tests/util/adls_util.py
File tests/util/adls_util.py:

http://gerrit.cloudera.org:8080/#/c/14311/3/tests/util/adls_util.py@34
PS3, Line 34: H
flake8: F821 undefined name 'HadoopFsCommandLineClient'



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0d45db1c00554e6fb6bcc0b552596d86d4e30144
Gerrit-Change-Number: 14311
Gerrit-PatchSet: 3
Gerrit-Owner: Sahil Takiar <stak...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com>
Gerrit-Comment-Date: Fri, 27 Sep 2019 01:10:25 +0000
Gerrit-HasComments: Yes

Reply via email to