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

Change subject: PREVIEW: IMPALA-7290: part 2: Add HS2 support to Impala shell
......................................................................


Patch Set 10:

(211 comments)

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/__init__.py
File shell/ext-py/bitarray-0.9.0/bitarray/__init__.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/__init__.py@11
PS10, Line 11: from bitarray._bitarray import _bitarray, bitdiff, bits2bytes, 
_sysinfo
flake8: F401 'bitarray._bitarray._sysinfo' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/__init__.py@11
PS10, Line 11: from bitarray._bitarray import _bitarray, bitdiff, bits2bytes, 
_sysinfo
flake8: F401 'bitarray._bitarray.bits2bytes' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/__init__.py@11
PS10, Line 11: from bitarray._bitarray import _bitarray, bitdiff, bits2bytes, 
_sysinfo
flake8: F401 'bitarray._bitarray.bitdiff' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py
File shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@21
PS10, Line 21: from bitarray import bitarray, bitdiff, bits2bytes, __version__
flake8: E402 module level import not at top of file


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@84
PS10, Line 84:
flake8: E221 multiple spaces before operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@85
PS10, Line 85:
flake8: E221 multiple spaces before operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@127
PS10, Line 127: class TestsModuleFunctions(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@167
PS10, Line 167: +
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@167
PS10, Line 167: +
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@167
PS10, Line 167: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@167
PS10, Line 167: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@168
PS10, Line 168: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@168
PS10, Line 168: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@176
PS10, Line 176: class CreateObjectTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@241
PS10, Line 241: :
flake8: E231 missing whitespace after ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@282
PS10, Line 282:
flake8: E261 at least two spaces before inline comment


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@309
PS10, Line 309: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@314
PS10, Line 314: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@321
PS10, Line 321: +
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@331
PS10, Line 331: class ToObjectsTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@360
PS10, Line 360: class MetaDataTests(unittest.TestCase):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@396
PS10, Line 396: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@404
PS10, Line 404: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@415
PS10, Line 415: class SliceTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@419
PS10, Line 419:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@422
PS10, Line 422:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@427
PS10, Line 427:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@434
PS10, Line 434: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@435
PS10, Line 435:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@474
PS10, Line 474:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@511
PS10, Line 511:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@535
PS10, Line 535: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@563
PS10, Line 563: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@570
PS10, Line 570:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@596
PS10, Line 596: class MiscTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@644
PS10, Line 644:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@644
PS10, Line 644:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@646
PS10, Line 646:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@646
PS10, Line 646:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@665
PS10, Line 665: +
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@757
PS10, Line 757: class SpecialMethodTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@767
PS10, Line 767:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@771
PS10, Line 771: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@779
PS10, Line 779:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@783
PS10, Line 783: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@797
PS10, Line 797: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@813
PS10, Line 813: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@849
PS10, Line 849: tests.append(SpecialMethodTests)
flake8: E305 expected 2 blank lines after class or function definition, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@853
PS10, Line 853: class NumberTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@875
PS10, Line 875: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@895
PS10, Line 895: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@920
PS10, Line 920: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@948
PS10, Line 948: class BitwiseTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@967
PS10, Line 967:
flake8: E261 at least two spaces before inline comment


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@972
PS10, Line 972:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@984
PS10, Line 984:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@994
PS10, Line 994:
flake8: E222 multiple spaces after operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1035
PS10, Line 1035: class SequenceTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1096
PS10, Line 1096: class ExtendTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1237
PS10, Line 1237: class MethodTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1252
PS10, Line 1252: +
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1254
PS10, Line 1254: +
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1257
PS10, Line 1257: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1277
PS10, Line 1277: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1336
PS10, Line 1336: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1356
PS10, Line 1356: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1366
PS10, Line 1366: *
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1389
PS10, Line 1389:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1389
PS10, Line 1389:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1390
PS10, Line 1390:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1390
PS10, Line 1390:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1391
PS10, Line 1391:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1391
PS10, Line 1391:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1392
PS10, Line 1392:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1392
PS10, Line 1392:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1393
PS10, Line 1393:
flake8: E241 multiple spaces after ','


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1401
PS10, Line 1401:
flake8: E261 at least two spaces before inline comment


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1410
PS10, Line 1410: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1438
PS10, Line 1438: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1438
PS10, Line 1438: *
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1439
PS10, Line 1439: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1442
PS10, Line 1442: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1467
PS10, Line 1467: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1489
PS10, Line 1489: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1501
PS10, Line 1501: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1526
PS10, Line 1526: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1541
PS10, Line 1541: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1553
PS10, Line 1553: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1561
PS10, Line 1561: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1575
PS10, Line 1575: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1603
PS10, Line 1603: class StringTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1637
PS10, Line 1637: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1660
PS10, Line 1660: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1691
PS10, Line 1691: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1716
PS10, Line 1716: class FileTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1726
PS10, Line 1726: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1755
PS10, Line 1755: ,
flake8: E401 multiple imports on one line


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1773
PS10, Line 1773: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1776
PS10, Line 1776:
flake8: E261 at least two spaces before inline comment


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1781
PS10, Line 1781: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1790
PS10, Line 1790: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1805
PS10, Line 1805: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1827
PS10, Line 1827: *
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1831
PS10, Line 1831: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1840
PS10, Line 1840: ;
flake8: E702 multiple statements on one line (semicolon)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1840
PS10, Line 1840:
flake8: E241 multiple spaces after ';'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1843
PS10, Line 1843: ;
flake8: E702 multiple statements on one line (semicolon)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1843
PS10, Line 1843:
flake8: E241 multiple spaces after ';'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1844
PS10, Line 1844: ;
flake8: E702 multiple statements on one line (semicolon)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1844
PS10, Line 1844:
flake8: E241 multiple spaces after ';'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1845
PS10, Line 1845: ;
flake8: E702 multiple statements on one line (semicolon)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1845
PS10, Line 1845:
flake8: E241 multiple spaces after ';'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1846
PS10, Line 1846: ;
flake8: E702 multiple statements on one line (semicolon)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1846
PS10, Line 1846:
flake8: E241 multiple spaces after ';'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1847
PS10, Line 1847: ;
flake8: E702 multiple statements on one line (semicolon)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1847
PS10, Line 1847:
flake8: E241 multiple spaces after ';'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1852
PS10, Line 1852: ;
flake8: E703 statement ends with a semicolon


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1859
PS10, Line 1859: ;
flake8: E703 statement ends with a semicolon


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1865
PS10, Line 1865: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1924
PS10, Line 1924: class PrefixCodeTests(unittest.TestCase, Util):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1930
PS10, Line 1930: :
flake8: E231 missing whitespace after ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1947
PS10, Line 1947: :
flake8: E231 missing whitespace after ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1983
PS10, Line 1983: d
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1988
PS10, Line 1988: :
flake8: E231 missing whitespace after ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@1989
PS10, Line 1989: :
flake8: E231 missing whitespace after ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2083
PS10, Line 2083:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2084
PS10, Line 2084:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2085
PS10, Line 2085:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2086
PS10, Line 2086:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2087
PS10, Line 2087:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2102
PS10, Line 2102:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2103
PS10, Line 2103:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2104
PS10, Line 2104:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2105
PS10, Line 2105:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2106
PS10, Line 2106:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2107
PS10, Line 2107:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2108
PS10, Line 2108:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2109
PS10, Line 2109:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2110
PS10, Line 2110:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2111
PS10, Line 2111: 
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2112
PS10, Line 2112:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2113
PS10, Line 2113:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2114
PS10, Line 2114:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2115
PS10, Line 2115:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2116
PS10, Line 2116:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2117
PS10, Line 2117:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2118
PS10, Line 2118:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2119
PS10, Line 2119:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2120
PS10, Line 2120:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2121
PS10, Line 2121:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2122
PS10, Line 2122:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2123
PS10, Line 2123:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2124
PS10, Line 2124:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2125
PS10, Line 2125:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2126
PS10, Line 2126:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2127
PS10, Line 2127:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2128
PS10, Line 2128:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2129
PS10, Line 2129:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2145
PS10, Line 2145: class BufferInterfaceTests(unittest.TestCase):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2160
PS10, Line 2160:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2176
PS10, Line 2176: if sys.version_info[:2] >= (2, 7):
flake8: E305 expected 2 blank lines after class or function definition, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/bitarray/test_bitarray.py@2181
PS10, Line 2181: def run(verbosity=1, repeat=1):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py
File shell/ext-py/bitarray-0.9.0/setup.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@19
PS10, Line 19:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@19
PS10, Line 19:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@20
PS10, Line 20:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@20
PS10, Line 20:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@21
PS10, Line 21:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@21
PS10, Line 21:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@22
PS10, Line 22:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@22
PS10, Line 22:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@23
PS10, Line 23:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@23
PS10, Line 23:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@24
PS10, Line 24:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@24
PS10, Line 24:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@42
PS10, Line 42:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@42
PS10, Line 42:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@43
PS10, Line 43:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@43
PS10, Line 43:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@44
PS10, Line 44:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@44
PS10, Line 44:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@44
PS10, Line 44:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@44
PS10, Line 44:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@45
PS10, Line 45:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/ext-py/bitarray-0.9.0/setup.py@45
PS10, Line 45:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py
File shell/impala_client.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@31
PS10, Line 31: from ImpalaService.ImpalaService import TPingImpalaServiceResp
flake8: F401 'ImpalaService.ImpalaService.TPingImpalaServiceResp' imported but 
unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@36
PS10, Line 36: from TCLIService import TCLIService
flake8: F401 'TCLIService.TCLIService' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@37
PS10, Line 37: from TCLIService.TCLIService import (TExecuteStatementReq, 
TOpenSessionReq,
flake8: F401 'TCLIService.TCLIService.TGetInfoReq' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@37
PS10, Line 37: from TCLIService.TCLIService import (TExecuteStatementReq, 
TOpenSessionReq,
flake8: F401 'TCLIService.TCLIService.TGetInfoType' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@82
PS10, Line 82: class QueryOptionLevels:
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@120
PS10, Line 120: class MissingThriftMethodException(Exception):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@124
PS10, Line 124: d
flake8: E301 expected 1 blank line, found 0


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@372
PS10, Line 372: d
flake8: E501 line too long (94 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@373
PS10, Line 373: t
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@380
PS10, Line 380: p
flake8: E501 line too long (97 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@389
PS10, Line 389: f
flake8: E501 line too long (100 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@549
PS10, Line 549: class ImpalaHS2Client(ImpalaClient):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@779
PS10, Line 779: #
flake8: E265 block comment should start with '# '


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@849
PS10, Line 849: e
flake8: E722 do not use bare except'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@930
PS10, Line 930: )
flake8: E501 line too long (91 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@939
PS10, Line 939: )
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@961
PS10, Line 961: n
flake8: E714 test for object identity should be 'is not'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@979
PS10, Line 979: s
flake8: F821 undefined name 'status'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_client.py@1021
PS10, Line 1021: #
flake8: E265 block comment should start with '# '


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_shell.py
File shell/impala_shell.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_shell.py@38
PS10, Line 38: from impala_client import (ImpalaHS2Client, ImpalaBeeswaxClient, 
DisconnectedException,
flake8: F401 'impala_client.TApplicationException' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_shell.py@600
PS10, Line 600:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_shell.py@828
PS10, Line 828:
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_shell.py@829
PS10, Line 829: l
flake8: E501 line too long (93 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/impala_shell.py@1753
PS10, Line 1753: ,
flake8: E401 multiple imports on one line


http://gerrit.cloudera.org:8080/#/c/12884/10/shell/option_parser.py
File shell/option_parser.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/shell/option_parser.py@262
PS10, Line 262: "
flake8: E501 line too long (91 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12884/10/tests/shell/test_shell_commandline.py
File tests/shell/test_shell_commandline.py:

http://gerrit.cloudera.org:8080/#/c/12884/10/tests/shell/test_shell_commandline.py@20
PS10, Line 20: import shlex
flake8: F401 'shlex' imported but unused


http://gerrit.cloudera.org:8080/#/c/12884/10/tests/shell/test_shell_commandline.py@890
PS10, Line 890: d
flake8: E501 line too long (94 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6d5cc83d545aacc659523f29b1d6feed672e2a12
Gerrit-Change-Number: 12884
Gerrit-PatchSet: 10
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Comment-Date: Tue, 23 Apr 2019 17:17:30 +0000
Gerrit-HasComments: Yes

Reply via email to