You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by sounakr <gi...@git.apache.org> on 2017/12/05 14:17:49 UTC

[GitHub] carbondata pull request #1595: [WIP][Spark-2.2]Carbon-Spark2.2 Alter Add | M...

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

    https://github.com/apache/carbondata/pull/1595#discussion_r154956857
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala ---
    @@ -155,9 +153,12 @@ class CarbonFileMetastore extends CarbonMetaStore {
               case Some(name) if name.equals("org.apache.spark.sql.CarbonSource") => name
               case _ => throw new NoSuchTableException(database, tableIdentifier.table)
             }
    -        new CarbonSource().createRelation(sparkSession.sqlContext,
    -          catalogTable.storage.properties
    -        ).asInstanceOf[CarbonDatasourceHadoopRelation].carbonRelation
    +        val identifier: AbsoluteTableIdentifier = AbsoluteTableIdentifier.from(
    --- End diff --
    
    This code is added to prevent infinite loop which occured due to TablePath changes. @ravipesala  Please have a look. 


---