Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/21628 )
Change subject: IMPALA-12907: Add testcases for TPC-H/TPC-DS queries with tuple caching ...................................................................... Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py File tests/query_test/test_tuple_cache_tpc_queries.py: http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py@20 PS2, Line 20: from builtins import range flake8: F401 'builtins.range' imported but unused http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py@21 PS2, Line 21: import os flake8: F401 'os' imported but unused http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py@39 PS2, Line 39: @SkipIf.not_tuple_cache flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py@51 PS2, Line 51: a flake8: W504 line break after binary operator http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py@59 PS2, Line 59: @SkipIf.not_tuple_cache flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/21628/2/tests/query_test/test_tuple_cache_tpc_queries.py@71 PS2, Line 71: a flake8: W504 line break after binary operator -- To view, visit http://gerrit.cloudera.org:8080/21628 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I967372744d8dda25cbe372aefec04faec5a76847 Gerrit-Change-Number: 21628 Gerrit-PatchSet: 2 Gerrit-Owner: Yida Wu <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Kurt Deschler <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Thu, 01 Aug 2024 06:39:24 +0000 Gerrit-HasComments: Yes
