Zoltan Ivanfi has posted comments on this change. Change subject: IMPALA-784: Use `-s in SHOW CREATE TABLE output ......................................................................
Patch Set 1: The functional tests still fail with false positives like - CREATE TABLE functional_kudu.dimtbl (id BIGINT, name STRING, zip INT) + CREATE TABLE `functional_kudu`.`dimtbl` (`id` BIGINT, `name` STRING, `zip` INT) I haven't fixed these yet, because I still have to figure out where the expected strings come from, but first I wanted to make sure that we agree on the approach to be taken. The same applies to the overly long lines in the unit test. That test is fixed functionally but not stylistically, as I didn't want to spend too much time on it until I get feedback about whether this is the right approach. Thanks, Zoltan -- To view, visit http://gerrit.cloudera.org:8080/4527 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8e66439daa35d2e3725e7277c16f3cc5fdb6f128 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Zoltan Ivanfi <z...@cloudera.com> Gerrit-Reviewer: Zoltan Ivanfi <z...@cloudera.com> Gerrit-HasComments: No