Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/19114 )
Change subject: IMPALA-11607: Support for TPC-H and TPC-DS test datasets stored as Iceberg tables ...................................................................... Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py File testdata/bin/generate-schema-statements.py: http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py@762 PS3, Line 762: # Iceberg tables do not support SMALLINT type flake8: E101 indentation contains mixed spaces and tabs http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py@762 PS3, Line 762: # Iceberg tables do not support SMALLINT type flake8: W191 indentation contains tabs http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py@762 PS3, Line 762: # Iceberg tables do not support SMALLINT type tab used for whitespace http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py@764 PS3, Line 764: # Constraint does not make sense for Iceberg tables flake8: E101 indentation contains mixed spaces and tabs http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py@764 PS3, Line 764: # Constraint does not make sense for Iceberg tables flake8: W191 indentation contains tabs http://gerrit.cloudera.org:8080/#/c/19114/3/testdata/bin/generate-schema-statements.py@764 PS3, Line 764: # Constraint does not make sense for Iceberg tables tab used for whitespace -- To view, visit http://gerrit.cloudera.org:8080/19114 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic8bbc5209e7e649f67a48144a2844b35d9f9c7f1 Gerrit-Change-Number: 19114 Gerrit-PatchSet: 3 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Sun, 09 Oct 2022 08:51:49 +0000 Gerrit-HasComments: Yes
