Mailing-List: contact issues-help@carbondata.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@carbondata.apache.org
From: ravipesala <git@git.apache.org>
To: issues@carbondata.apache.org
Reply-To: issues@carbondata.apache.org
References: <git-pr-1476-carbondata@git.apache.org>
In-Reply-To: <git-pr-1476-carbondata@git.apache.org>
Subject: [GitHub] carbondata pull request #1476: [CARBONDATA-1527] [CARBONDATA-1528] [PreAgg] ...
Content-Type: text/plain
Message-Id: <20171113100838.C7AD0DFBC7@git1-us-west.apache.org>
Date: Mon, 13 Nov 2017 10:08:38 +0000 (UTC)
archived-at: Mon, 13 Nov 2017 10:08:40 -0000

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

    https://github.com/apache/carbondata/pull/1476#discussion_r150493082
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateListeners.scala ---
    @@ -79,3 +79,190 @@ object LoadPostAggregateListener extends OperationEventListener {
         }
       }
     }
    +
    +object PreAggregateDataTypeChangePreListener extends OperationEventListener {
    +  /**
    +   * Called on a specified event occurrence
    +   *
    +   * @param event
    +   * @param operationContext
    +   */
    +  override def onEvent(event: Event, operationContext: OperationContext): Unit = {
    +    val dataTypeChangePreListener = event.asInstanceOf[AlterTableDataTypeChangePreEvent]
    +    val carbonTable = dataTypeChangePreListener.carbonTable
    +    val alterTableDataTypeChangeModel = dataTypeChangePreListener.alterTableDataTypeChangeModel
    +    val columnToBeAltered: String = alterTableDataTypeChangeModel.columnName
    +    val dataMapSchemas = carbonTable.getTableInfo.getDataMapSchemaList
    +    if (dataMapSchemas != null && !dataMapSchemas.isEmpty) {
    +      dataMapSchemas.asScala.foreach {
    --- End diff --
    
    move to above line


---