Sudhanshu Arora has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13091
Change subject: [acid] Predicate to test if dir must be included. ...................................................................... [acid] Predicate to test if dir must be included. The predicate is currently not set anywhere but the plan is to modify HdfsTable and use the validWriteIdList there in the predicate implementation and remove the directories that should not be used. Testing Done: - mvn test -Dtest=RecursingIteratorTest Added RecursingIteratorTest. Could have also used FileMetadataLoaderTest but oh! well Change-Id: If0e88281d277127c9499d37b95fbba55dcc7761c --- M fe/src/main/java/org/apache/impala/catalog/FileMetadataLoader.java M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java M fe/src/main/java/org/apache/impala/catalog/local/DirectMetaProvider.java M fe/src/main/java/org/apache/impala/common/FileSystemUtil.java M fe/src/test/java/org/apache/impala/catalog/FileMetadataLoaderTest.java M fe/src/test/java/org/apache/impala/catalog/HdfsPartitionTest.java A fe/src/test/java/org/apache/impala/common/RecursingIteratorTest.java A fe/src/test/java/org/apache/impala/planner/ExplainTest.java 8 files changed, 418 insertions(+), 20 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/91/13091/2 -- To view, visit http://gerrit.cloudera.org:8080/13091 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: If0e88281d277127c9499d37b95fbba55dcc7761c Gerrit-Change-Number: 13091 Gerrit-PatchSet: 2 Gerrit-Owner: Sudhanshu Arora <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Reviewer: Yongzhi Chen <[email protected]>
