Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24296 )

Change subject: WIP IMPALA-14985: Cleanup tests with unused 'vector' in EE tests
......................................................................


Patch Set 6: Code-Review+1

(2 comments)

http://gerrit.cloudera.org:8080/#/c/24296/6/tests/metadata/test_last_ddl_time_update.py
File tests/metadata/test_last_ddl_time_update.py:

http://gerrit.cloudera.org:8080/#/c/24296/6/tests/metadata/test_last_ddl_time_update.py@32
PS6, Line 32:     if cls.exploration_strategy() != 'exhaustive' and not IS_S3:
nit: could probably be a @SkipIf condition.


http://gerrit.cloudera.org:8080/#/c/24296/6/tests/query_test/test_io_metrics.py
File tests/query_test/test_io_metrics.py:

http://gerrit.cloudera.org:8080/#/c/24296/6/tests/query_test/test_io_metrics.py@53
PS6, Line 53:         query_options={'num_nodes': 1})
Does this effectively change the test?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8e2a8fe333e33a906178a0821eb554890be26132
Gerrit-Change-Number: 24296
Gerrit-PatchSet: 6
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Fri, 15 May 2026 15:30:19 +0000
Gerrit-HasComments: Yes

Reply via email to