[GitHub] carbondata pull request #1584: [CARBONDATA-1827] Added S3 Implementation

Posted by qiuchenjian-2 on
URL: http://apache-carbondata-dev-mailing-list-archive.168.s1.nabble.com/GitHub-carbondata-pull-request-1584-CARBONDATA-1827-Added-S3-implementation-and-tp28501p28551.html

Github user mohammadshahidkhan commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1584#discussion_r153692578
 
    --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/impl/FileFactory.java ---
    @@ -81,24 +77,22 @@ public static FileHolder getFileHolder(FileType fileType) {
           case HDFS:
           case ALLUXIO:
           case VIEWFS:
    -      case S3:
             return new DFSFileHolderImpl();
    +      case S3:
    +        return new S3FileHolderImpl();
           default:
             return new FileHolderImpl();
         }
       }
     
       public static FileType getFileType(String path) {
    -    String lowerPath = path.toLowerCase();
    --- End diff --
   
    Any reason why converting to lower case is removed.


---