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

    https://github.com/apache/carbondata/pull/2103#discussion_r179950250
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonSessionCatalog.scala
 ---
    @@ -63,5 +67,39 @@ trait CarbonSessionCatalog {
       /**
        * Update the storageformat with new location information
        */
    -  def updateStorageLocation(path: Path, storage: CatalogStorageFormat): 
CatalogStorageFormat
    +  def updateStorageLocation(
    +      path: Path,
    +      storage: CatalogStorageFormat,
    +      newTableName: String,
    +      dbName: String): CatalogStorageFormat
    +
    +  /**
    +   * Method used to update the table name
    +   * @param tableRenameVo
    +   */
    +  def alterTableRename(tableRenameVo: TableRenameVo): Unit
    --- End diff --
    
    please do not use VO object but put 3 parameters


---

Reply via email to