Paul Rogers has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12437
Change subject: IMPALA-8185: Abstract out real/mock file system operations ...................................................................... IMPALA-8185: Abstract out real/mock file system operations The recent addition of the test case builder introduced the idea of running a query in "test case mode" on a system different from the on on which the test case was created. A number of if-statements were used to mock file system operations which depend on the file system to exist. This patch abstracts out the mock vs. real operations into a set of "facade" classes. The HDFS facade performs the real opeations, the "mock" facade mocks the operations well enough to fool the planner. Selection of the facade depends on the recently-added test case mode query option. Tests: Reran all tests: both those that depend on a real file system and the test case builder tests that depend on the mock file system. Change-Id: I1a385923b64c9fb59cc6e700ee7ee14919398e6d --- M fe/src/main/java/org/apache/impala/analysis/Analyzer.java A fe/src/main/java/org/apache/impala/analysis/FileSystemFacade.java A fe/src/main/java/org/apache/impala/analysis/HdfsFileSystemFacade.java M fe/src/main/java/org/apache/impala/analysis/HdfsUri.java A fe/src/main/java/org/apache/impala/analysis/MockFileSystemFacade.java M fe/src/main/java/org/apache/impala/planner/HdfsPartitionPruner.java M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java 7 files changed, 282 insertions(+), 141 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/37/12437/1 -- To view, visit http://gerrit.cloudera.org:8080/12437 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I1a385923b64c9fb59cc6e700ee7ee14919398e6d Gerrit-Change-Number: 12437 Gerrit-PatchSet: 1 Gerrit-Owner: Paul Rogers <[email protected]>
