[email protected] has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24642 )

Change subject: IMPALA-15229: Derive the EC Policy column from file metadata, 
not RPCs
......................................................................


Patch Set 1:

(1 comment)

Responded to the auto-critic note on the .fbs change.

http://gerrit.cloudera.org:8080/#/c/24642/1/common/fbs/CatalogObjects.fbs
File common/fbs/CatalogObjects.fbs:

PS1:
> This file is used in communication between impalad and catalogd/statestore.
The new field follows these rules: it is appended after the last field with an
explicit id (8) and a default of 0, and no existing field is modified. Old
readers ignore the field; new readers see 0 for descriptors written by older
catalogd versions and fall back to the previous per-path lookup in that case.
This is covered in the commit message ("rolling upgrades keep working") and
'is_ec' is untouched.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I35fc413c4a1f4730baab55406b13f5541f379707
Gerrit-Change-Number: 24642
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Wed, 29 Jul 2026 11:59:59 +0000
Gerrit-HasComments: Yes

Reply via email to