impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4897: AnalysisException: specified cache pool does not exist
Date Tue, 14 Feb 2017 01:20:46 GMT
Sailesh Mukil has posted comments on this change.

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


Patch Set 3:

There were a few instances in the codebase where absolute paths were used for partitions and
they ended up getting stored in: [namenode]/[partition_name].

They could race if multiple tests end up using the same paths, causing more files to exist
than the test actually expects.

I've fixed those to paths relative to the table. Ran a private build and it passed too.

-- 
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: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <sailesh@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Sailesh Mukil <sailesh@cloudera.com>
Gerrit-HasComments: No

Mime
View raw message