Marcel Kornacker has posted comments on this change.

Change subject: Kudu: Remove TODO about checking for unsupported types in table 
loading
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2834/1/fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java
File fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java:

Line 154
> I think this line was added by Martin. The comments in the merge review are
do you want to assign this jira to yourself then? i'm fine with removing this 
todo, but if we know there's more work to be done, let's not lose sight of that 
(and an unassigned jira basically means it vanished :))


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I681d80776f28904a6171cac5aebde3a02327b295
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <[email protected]>
Gerrit-Reviewer: Casey Ching <[email protected]>
Gerrit-Reviewer: Marcel Kornacker <[email protected]>
Gerrit-HasComments: Yes

Reply via email to