impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lars Volker (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4615: Fix create table.sql command order
Date Wed, 08 Mar 2017 22:11:52 GMT
Lars Volker has posted comments on this change.

Change subject: IMPALA-4615: Fix create_table.sql command order
......................................................................


Patch Set 1:

(1 comment)

Thanks for the review. I updated the commit message.

http://gerrit.cloudera.org:8080/#/c/6317/1//COMMIT_MSG
Commit Message:

Line 13: files in those partitions and will fail to move the new data in place.
> How did you test this change?
Added an explanation to the commit message. I'm not sure how to test this on every run of
the test suite, but since we've been hitting it from time to time I think we should be ok.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f68eeb75ba2f43b96b8f3d82f902e291d3bd396
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Lars Volker <lv@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Lars Volker <lv@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message