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

Change subject: IMPALA-14848: Enable Puffin stats reading by default
......................................................................

IMPALA-14848: Enable Puffin stats reading by default

The initial patches that added support for Puffin stats disabled reading
them by default, because
* it only worked if the Puffin file was associated with the current
  snapshot
* Puffin stats always took precedence over HMS stats.

Since then both of the above issues have been resolved (IMPALA-13594,
IMPALA-14261). Now it is time to make reading Puffin stats the default.

So far Puffin stats tests were custom cluster tests, this change moves
most of them to test_iceberg.py.

The patch also updates the docs about Puffin stats.

Generated-by: Claude Opus 4.6

Change-Id: Ic634aeb50ac099fe31fe83ff6c06676ca6a5a46a
Reviewed-on: http://gerrit.cloudera.org:8080/24286
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M be/src/catalog/catalog.cc
M docs/topics/impala_iceberg.xml
M tests/custom_cluster/test_iceberg_with_puffin.py
M tests/query_test/test_iceberg.py
4 files changed, 267 insertions(+), 279 deletions(-)

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

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic634aeb50ac099fe31fe83ff6c06676ca6a5a46a
Gerrit-Change-Number: 24286
Gerrit-PatchSet: 3
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>

Reply via email to