trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sureshsubbiah <...@git.apache.org>
Subject [GitHub] trafodion pull request #1626: [TRAFODION-3126] Refactored HDFS client implem...
Date Tue, 03 Jul 2018 00:24:40 GMT
Github user sureshsubbiah commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1626#discussion_r199655367
  
    --- Diff: core/sql/src/main/java/org/trafodion/sql/HDFSClient.java ---
    @@ -126,6 +128,16 @@
           catch (IOException ioe) {
              throw new RuntimeException("Exception in HDFSClient static block", ioe);
           }
    +      try {
    +         boolean alluxioFs = defaultFs_ instanceof alluxio.hadoop.FileSystem;
    +      }
    +      catch (Throwable rte)
    +      {
    +         // Ignore the exception. It is not needed for alluxio to be installed
    +         // for the methods of this class to work if 
    +         // alluxio filesystem is NOT required
    +         alluxioNotInstalled_ = true;
    --- End diff --
    
    Can you please explain what error we get if the file to be read is an alluxio file and
alluxio is not installed? 


---

Mime
View raw message