carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sounakr <...@git.apache.org>
Subject [GitHub] carbondata pull request #1434: [CARBONDATA-1593]Add partition to table cause...
Date Fri, 27 Oct 2017 10:43:22 GMT
Github user sounakr commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1434#discussion_r147375453
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/partition/AlterTableSplitCarbonPartitionCommand.scala
---
    @@ -109,7 +109,7 @@ case class AlterTableSplitCarbonPartitionCommand(
         CarbonUtil.writeThriftTableToSchemaFile(schemaFilePath, thriftTable)
         // update the schema modified time
         carbonMetaStore.updateAndTouchSchemasUpdatedTime(storePath)
    -    sparkSession.catalog.refreshTable(tableName)
    +    sparkSession.catalog.refreshTable(s"$dbName.$tableName")
    --- End diff --
    
    Better to call SessionCatalog.refreshTable rather than Catalog refreshTable as it will
honour the DBName.
    sparkSession.sessionState.catalog.refreshTable() and pass TableIdentifier. 


---

Mime
View raw message