[GitHub] kevinjmh commented on a change in pull request #3106: [CARBONDATA-3276] Compacting table that do not exist should throw NoSuchTableException instead of MalformedCarbonCommandException

[GitBox <gi...@apache.org>]

kevinjmh commented on a change in pull request #3106: [CARBONDATA-3276] Compacting table that do not exist should throw NoSuchTableException instead of MalformedCarbonCommandException
URL: https://github.com/apache/carbondata/pull/3106#discussion_r251656896
 
 

 ##########
 File path: integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/DDLStrategy.scala
 ##########
 @@ -180,7 +182,8 @@ class DDLStrategy(sparkSession: SparkSession) extends SparkStrategy {
           Nil
           // TODO: remove this else check once the 2.1 version is unsupported by carbon
         } else {
-          throw new MalformedCarbonCommandException("Unsupported alter operation on hive table")
+          throw new MalformedCarbonCommandException(
+            "Unsupported alter add column operation on version 2.1")
 
 Review comment:
   Spark version 2.1? That should be clarified 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services