[ 
https://issues.apache.org/jira/browse/IMPALA-14848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18080435#comment-18080435
 ] 

ASF subversion and git services commented on IMPALA-14848:
----------------------------------------------------------

Commit 3b70c80db55b1d091fd6b4f564726454fdd73854 in impala's branch 
refs/heads/master from Zoltan Borok-Nagy
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=3b70c80db ]

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]>


> Enable Puffin stats reading by default
> --------------------------------------
>
>                 Key: IMPALA-14848
>                 URL: https://issues.apache.org/jira/browse/IMPALA-14848
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Catalog
>            Reporter: Zoltán Borók-Nagy
>            Assignee: Zoltán Borók-Nagy
>            Priority: Major
>              Labels: impala-iceberg
>
> The initial patch that added support for Puffin stats disabled reading them 
> by default, because
> * it only worked if the Puffin file was associated with the current snapshots
> * 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 Puffin stats reading default.
> Currently the Puffin stats tests are custom cluster tests, with this change 
> they can be moved to test_iceberg.py.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to