Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21866 )

Change subject: IMPALA-13415: Add a special testing mode to track Calcite 
progress
......................................................................


Patch Set 9:

(5 comments)

Example run using a single commit (i.e. no diff): 
https://jenkins.impala.io/view/all/job/calcite-report-prototype/15/artifact/Impala/calcite_report/html/index.html

http://gerrit.cloudera.org:8080/#/c/21866/7/tests/common/impala_test_suite.py
File tests/common/impala_test_suite.py:

http://gerrit.cloudera.org:8080/#/c/21866/7/tests/common/impala_test_suite.py@757
PS7, Line 757:
> flake8: E501 line too long (91 > 90 characters)
Done


http://gerrit.cloudera.org:8080/#/c/21866/7/tests/common/impala_test_suite.py@793
PS7, Line 793:
> flake8: E501 line too long (92 > 90 characters)
Done


http://gerrit.cloudera.org:8080/#/c/21866/7/tests/common/impala_test_suite.py@826
PS7, Line 826:
> flake8: E501 line too long (92 > 90 characters)
Done


http://gerrit.cloudera.org:8080/#/c/21866/7/tests/conftest.py
File tests/conftest.py:

http://gerrit.cloudera.org:8080/#/c/21866/7/tests/conftest.py@181
PS7, Line 181:
> flake8: E501 line too long (91 > 90 characters)
Done


http://gerrit.cloudera.org:8080/#/c/21866/7/tests/conftest.py@187
PS7, Line 187:
> flake8: E124 closing bracket does not match visual indentation
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I453c219c22b6cbc253574e0467d2c0d7b1fac092
Gerrit-Change-Number: 21866
Gerrit-PatchSet: 9
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
Gerrit-Comment-Date: Fri, 01 Nov 2024 21:09:11 +0000
Gerrit-HasComments: Yes

Reply via email to