Michael Brown has posted comments on this change. Change subject: Use unique_database fixture in test_col_stats.py. ......................................................................
Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/2963/1/tests/metadata/test_compute_stats.py File tests/metadata/test_compute_stats.py: Line 148: self.client.execute("insert into table %s select cast(int_col as string) "\ : "from functional.alltypes limit 10" % table_name) Backslash not needed when a multi-line string literal is within parentheses. Line 153: Why did you remove the REFRESH that existed in the test before? Line 158: self.client.execute("invalidate metadata %s" % table_name) Why did you add the invalidate metadata that was not in the test before? Line 168: self.client.execute("compute stats %s" % table_name) Another REFRESH removed after L168 (see L79-80 in the removed file). -- To view, visit http://gerrit.cloudera.org:8080/2963 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I88aa77464a95993c018e19a52eeb496d7c3eef08 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Alex Behm <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-HasComments: Yes
