Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13410 )

Change subject: hms: followup to 6a8d1d2
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/13410/1/src/kudu/tools/tool_action_hms.cc
File src/kudu/tools/tool_action_hms.cc:

http://gerrit.cloudera.org:8080/#/c/13410/1/src/kudu/tools/tool_action_hms.cc@307
PS1, Line 307:       // If this is a non-legacy, managed table, we expect a 
table ID to exist
> Can you adjust this to match the logic below more closely. Something like:
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0d624258e1d177363fceba6b40d6aee77b142b54
Gerrit-Change-Number: 13410
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Comment-Date: Thu, 23 May 2019 17:33:56 +0000
Gerrit-HasComments: Yes

Reply via email to