Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22914 )

Change subject: IMPALA-14081: Support create/drop paimon table for impala
......................................................................


Patch Set 14:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22914/14/common/thrift/CatalogObjects.thrift
File common/thrift/CatalogObjects.thrift:

http://gerrit.cloudera.org:8080/#/c/22914/14/common/thrift/CatalogObjects.thrift@712
PS14, Line 712:   // Paimon file system table location
              :   1: required binary tbl_obj
Why this is binary?
I see this is a serialized version of FePaimonTable. Can it be represented as 
collection of fields instead of single serialized binary?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I57e77f28151e4a91353ef77050f9f0cd7d9d05ef
Gerrit-Change-Number: 22914
Gerrit-PatchSet: 14
Gerrit-Owner: ji chen <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: ji chen <[email protected]>
Gerrit-Comment-Date: Tue, 26 Aug 2025 20:22:34 +0000
Gerrit-HasComments: Yes

Reply via email to