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 <d...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to