Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21971 )

Change subject: IMPALA-13403: Trivial changes in StorageDescriptor of 
ALTER_TABLE event is not enough to decide file metadata reload can be skipped
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/21971/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/21971/3//COMMIT_MSG@7
PS3, Line 7: IMPALA-13403: Trivial changes in StorageDescriptor of ALTER_TABLE
           : event is not enough to decide file metadata reload can be skipped
nit: Can we use a shorter title, e.g. "refactor the checks of skip reloading 
file metadata for ALTER_TABLE events"?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia969dd32385ac5a1a9a65890a5ccc8cd257f4b97
Gerrit-Change-Number: 21971
Gerrit-PatchSet: 3
Gerrit-Owner: Sai Hemanth Gantasala <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Mon, 11 Nov 2024 01:04:05 +0000
Gerrit-HasComments: Yes

Reply via email to