carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jackylk <...@git.apache.org>
Subject [GitHub] carbondata pull request #1694: [CARBONDATA-1925][Pre-Aggregate]Added code to...
Date Thu, 28 Dec 2017 11:24:13 GMT
Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1694#discussion_r158930929
  
    --- Diff: integration/spark-common/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchemaCommon.scala
---
    @@ -467,10 +474,13 @@ class TableNewProcessor(cm: TableModel) {
         // Sort columns should be at the begin of all columns
         cm.sortKeyDims.get.foreach { keyDim =>
           val field = cm.dimCols.find(keyDim equals _.column).get
    -      val encoders = if (cm.parentTable.isDefined && cm.dataMapRelation.get.get(field).isDefined)
{
    +      val encoders = if (cm.parentTable.isDefined &&
    +                         cm.dataMapRelation.get.get(field).isDefined &&
    +                         cm.dataMapRelation.get.get(field).get.columnTableRelationList.size==1
) {
    --- End diff --
    
    add space before and after `=`


---

Mime
View raw message