Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14714 )
Change subject: IMPALA-8705: ISO:SQL:2016 datetime patterns - Milestone 3 ...................................................................... Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py File tests/query_test/test_cast_with_format.py: http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py@1565 PS8, Line 1565: >>>>>>> IMPALA-8705: ISO:SQL:2016 datetime patterns - Milestone 3 flake8: E305 expected 2 blank lines after class or function definition, found 0 http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py@1565 PS8, Line 1565: > flake8: E999 SyntaxError: invalid syntax http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py@1565 PS8, Line 1565: > flake8: E225 missing whitespace around operator http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py@1565 PS8, Line 1565: - flake8: E226 missing whitespace around arithmetic operator http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py@1565 PS8, Line 1565: : flake8: E231 missing whitespace after ':' http://gerrit.cloudera.org:8080/#/c/14714/8/tests/query_test/test_cast_with_format.py@1565 PS8, Line 1565: : flake8: E231 missing whitespace after ':' -- To view, visit http://gerrit.cloudera.org:8080/14714 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic797f19a1311b54e5d00d01d0a7afe1f0f21fb8f Gerrit-Change-Number: 14714 Gerrit-PatchSet: 8 Gerrit-Owner: Gabor Kaszab <gaborkas...@cloudera.com> Gerrit-Reviewer: Attila Jeges <atti...@cloudera.com> Gerrit-Reviewer: Gabor Kaszab <gaborkas...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Comment-Date: Wed, 04 Dec 2019 11:01:48 +0000 Gerrit-HasComments: Yes