[Impala-CR](cdh5-trunk) IMPALA-3491: Use unique database fixture in test metadata query statements.py.

["Alex Behm (Code Review)" <ge...@cloudera.org>]

Alex Behm has posted comments on this change.

Change subject: IMPALA-3491: Use unique_database fixture in test_metadata_query_statements.py.
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3103/1/tests/metadata/test_metadata_query_statements.py
File tests/metadata/test_metadata_query_statements.py:

Line 169
> Did you drop this because self.cleanup_db already does it, or because it's 
As far as I could tell, this part is only meant for cleaning up before the test, so it doesn't matter whether Hive or Impala do it. Having Impala do it makes things easier (no need for invalidate metadata). So yes, cleanup_db() does it.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia0af1dc6534234508bd0fed03531f7fe8ff556aa
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Alex Behm <al...@cloudera.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jb...@cloudera.com>
Gerrit-HasComments: Yes