impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marcel Kornacker (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) Kudu: Remove TODO about checking for unsupported types in table loading
Date Thu, 28 Apr 2016 22:47:57 GMT
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 <casey@cloudera.com>
Gerrit-Reviewer: Casey Ching <casey@cloudera.com>
Gerrit-Reviewer: Marcel Kornacker <marcel@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message