[GitHub] carbondata pull request #2006: Fixed rename table with partitions not workin...

classic Classic list List threaded Threaded
27 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #2006: [CARBONDATA-2209] Fixed rename table with par...

qiuchenjian-2
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2006#discussion_r171867026
 
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/CarbonAlterTableRenameCommand.scala ---
    @@ -138,6 +147,27 @@ private[sql] case class CarbonAlterTableRenameCommand(
               sys.error(s"Folder rename failed for table $oldDatabaseName.$oldTableName")
             }
           }
    +      val updatedParts = updatePartitionLocations(
    --- End diff --
   
    ok


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2006: [CARBONDATA-2209] Fixed rename table with partitions...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2006
 
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/4047/



---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2006: [CARBONDATA-2209] Fixed rename table with partitions...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2006
 
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/2801/



---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2006: [CARBONDATA-2209] Fixed rename table with partitions...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2006
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/4048/



---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2006: [CARBONDATA-2209] Fixed rename table with partitions...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2006
 
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/2802/



---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2006: [CARBONDATA-2209] Fixed rename table with partitions...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user gvramana commented on the issue:

    https://github.com/apache/carbondata/pull/2006
 
    LGTM


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #2006: [CARBONDATA-2209] Fixed rename table with par...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user asfgit closed the pull request at:

    https://github.com/apache/carbondata/pull/2006


---
12