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

    https://github.com/apache/carbondata/pull/2627#discussion_r210203693
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/DDLStrategy.scala
 ---
    @@ -277,6 +278,10 @@ class DDLStrategy(sparkSession: SparkSession) extends 
SparkStrategy {
                   throw new MalformedCarbonCommandException(
                     "Streaming property value is incorrect")
                 }
    +            if (CarbonTable.hasMVDataMap(carbonTable)) {
    +              throw new MalformedCarbonCommandException(
    +                "The table which has MV datamap, does not support set 
streaming property")
    --- End diff --
    
    Please optimize the error message to
    ```
    Cannot set streaming property on table which has MV datamap
    ```
    OR
    just remove the comma in your original error message.


---

Reply via email to