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

Posted to issues@carbondata.apache.org by kumarvishal09 <gi...@git.apache.org> on 2018/01/03 10:09:21 UTC

[GitHub] carbondata pull request #1697: [CARBONDATA-1719][Pre-Aggregate][Bug] Fixed b...

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

    https://github.com/apache/carbondata/pull/1697#discussion_r159391691
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateListeners.scala ---
    @@ -161,7 +163,8 @@ object PreAggregateDataTypeChangePreListener extends OperationEventListener {
         }
         if (carbonTable.isChildDataMap) {
           throw new UnsupportedOperationException(
    -        s"Cannot change data type for columns in pre-aggregate table ${ carbonTable.getDatabaseName
    +        s"Cannot change data type for columns in pre-aggregate table ${
    --- End diff --
    
    Please avoid unnecessary changes.


---