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

Change subject: IMPALA-8450: Add support for zstd and lz4 in parquet
......................................................................


Patch Set 6:

(13 comments)

http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py
File tests/query_test/test_insert_parquet.py:

http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@182
PS6, Line 182:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@183
PS6, Line 183:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@185
PS6, Line 185:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@187
PS6, Line 187:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@188
PS6, Line 188:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@190
PS6, Line 190:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@191
PS6, Line 191:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@193
PS6, Line 193:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@195
PS6, Line 195:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@196
PS6, Line 196:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@198
PS6, Line 198:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@199
PS6, Line 199:
flake8: E203 whitespace before ','


http://gerrit.cloudera.org:8080/#/c/13396/6/tests/query_test/test_insert_parquet.py@201
PS6, Line 201:
flake8: E203 whitespace before ','



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I98c6dcf3d0a873380e4fa4cf03eb7e924e4ee768
Gerrit-Change-Number: 13396
Gerrit-PatchSet: 6
Gerrit-Owner: Abhishek Rawat <ara...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Wed, 22 May 2019 00:11:21 +0000
Gerrit-HasComments: Yes

Reply via email to