Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16628 )
Change subject: IMPALA-10266: Identify FileSystem type based on the protocol scheme. ...................................................................... IMPALA-10266: Identify FileSystem type based on the protocol scheme. Frontend identifies the type of FileSystem in two ways. The first is done using the instanceof operator with subclasses of org.apache.hadoop.fs.FileSystem. The second is by checking the FileSystem protocol scheme. This patch standardizes the FileSystem identification based on the scheme only. Testing: - Add several tests in FileSystemUtilTest to check validity of some FileSystemUtil functions. - Run and pass core tests. Change-Id: I04492326a6e84895eef369fc11a3ec11f1536b6b Reviewed-on: http://gerrit.cloudera.org:8080/16628 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M fe/src/main/java/org/apache/impala/analysis/LoadDataStmt.java M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/org/apache/impala/common/FileSystemUtil.java M fe/src/main/java/org/apache/impala/service/JniFrontend.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeStmtsTest.java M fe/src/test/java/org/apache/impala/common/FileSystemUtilTest.java 6 files changed, 265 insertions(+), 69 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16628 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I04492326a6e84895eef369fc11a3ec11f1536b6b Gerrit-Change-Number: 16628 Gerrit-PatchSet: 8 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>