carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From SangeetaGulia <...@git.apache.org>
Subject [GitHub] carbondata pull request #1787: [CARBONDATA-2017] Fix input path checking whe...
Date Thu, 11 Jan 2018 07:40:48 GMT
Github user SangeetaGulia commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1787#discussion_r160882178
  
    --- Diff: integration/spark-common/src/main/scala/org/apache/spark/util/FileUtils.scala
---
    @@ -73,7 +73,8 @@ object FileUtils {
           val stringBuild = new StringBuilder()
           val filePaths = inputPath.split(",")
           for (i <- 0 until filePaths.size) {
    -        val fileType = FileFactory.getFileType(filePaths(i))
    +        val filePath = CarbonUtil.checkAndAppendHDFSUrl(filePaths(i))
    --- End diff --
    
    @jackylk I have verified this. It is working fine with S3 also. We will now be able to
use the carbon property **carbon.ddl.base.hdfs.url** for s3 also to provide base URL.


---

Mime
View raw message