Sailesh Mukil has posted comments on this change.

Change subject: IMPALA-4897: AnalysisException: specified cache pool does not 
exist
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/5972/1/tests/metadata/test_ddl.py
File tests/metadata/test_ddl.py:

PS1, Line 258: if IS_HDFS:
             :       self.run_test_case('QueryTest/alter-table-hdfs-caching', 
vector,
             :           use_db=unique_database, 
multiple_impalad=self._use_multiple_impalad(vector))
> I think it may be best to put it in test_hdfs_caching.py? What do you think
I thought about that but the goal of this test is to test an ALTER TABLE 
functionality and the hdfs caching part is not the core of the test, which is 
why I left it here instead.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iefe61556bc28ae320f3f41fdc930d37b258d970a
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogian...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to