Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22931 )

Change subject: IMPALA-14023: Fix test_scan_metrics_in_profile in non-HDFS 
builds
......................................................................

IMPALA-14023: Fix test_scan_metrics_in_profile in non-HDFS builds

test_scan_metrics_in_profile was querying pre-written Iceberg V2
tables. The position delete files of such tables contain hard-coded
URIs of data files, i.e. URIs that start with "hdfs://localhost...".
Therefore the test only worked well in HDFS builds.

This patch splits the test into two parts:

* test_scan_metrics_in_profile_basic: it works on all storage systems
  as it only works on Iceberg tables that don't have delete files.
* test_scan_metrics_in_profile_with_deletes: uses Iceberg tables
  that have delete files, therefore it is only executed on HDFS.

Change-Id: I80a7c6469a7f56b58254e1327a05ef7b3dc9c9ff
Reviewed-on: http://gerrit.cloudera.org:8080/22931
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
A 
testdata/workloads/functional-query/queries/QueryTest/iceberg-scan-metrics-basic.test
R 
testdata/workloads/functional-query/queries/QueryTest/iceberg-scan-metrics-with-deletes.test
M tests/query_test/test_iceberg.py
3 files changed, 61 insertions(+), 55 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I80a7c6469a7f56b58254e1327a05ef7b3dc9c9ff
Gerrit-Change-Number: 22931
Gerrit-PatchSet: 4
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>

Reply via email to