Dan Burkert has posted comments on this change. Change subject: Add table id to AlterTableResponsePB ......................................................................
Patch Set 1: I'm using it in the rust client. There are a couple of cases where the field could be used in the C++ and Java clients: * When waiting for the table alteration to finish, the table ID can be set instead of the table name in IsAlterTableDoneRequestPB which closes a race between concurrent renames. * The table ID can be used to scope the clearing of the meta cache to just the altered table. I actually tried to implement both of these in the C++ client along with this patch, but they aren't exactly trivial changes. Partially clearing the meta-cache breaks some invariants between the internal cache maps, and there isn't currently a way to use table IDs instead of table names when doing RPCs (especially in the client-internal layer). -- To view, visit http://gerrit.cloudera.org:8080/3859 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I63402581fbdaef2abdf37910c1dfaedbdcba76e2 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: No
