[jira] [Commented] (CARBONDATA-268) CarbonOptimizer has performance problem

[Akash R Nilugal (Jira)]

    [ https://issues.apache.org/jira/browse/CARBONDATA-268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15516077#comment-15516077 ]

ASF GitHub Bot commented on CARBONDATA-268:
-------------------------------------------

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

    https://github.com/apache/incubator-carbondata/pull/190#discussion_r80219576
 
    --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/optimizer/CarbonOptimizer.scala ---
    @@ -87,20 +130,32 @@ class ResolveCarbonFunctions(relations: Seq[CarbonDecoderRelation])
           return plan
         }
         var decoder = false
    +    val mapOfNodes = new java.util.HashMap[LogicalPlan, ExtraNodeInfo]
    +    fillNodeInfo(plan, mapOfNodes)
         val aliasMap = CarbonAliasDecoderRelation()
         // collect alias information before hand.
         collectInformationOnAttributes(plan, aliasMap)
    -    val transFormedPlan =
    -      plan transformDown {
    -        case cd: CarbonDictionaryTempDecoder if cd.isOuter =>
    -          decoder = true
    -          cd
    +
    +    def hasCarbonRelation(currentPlan: LogicalPlan): Boolean = {
    +      val extraNodeInfo = mapOfNodes.get(currentPlan)
    --- End diff --
   
    Change the name of mapofNodes to mapOfNonCarbonPlanNodes

> CarbonOptimizer has performance problem
> ---------------------------------------
>
>                 Key: CARBONDATA-268
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-268
>             Project: CarbonData
>          Issue Type: Bug
>    Affects Versions: 0.1.0-incubating
>            Reporter: QiangCai
>
> 1. invoke relation lookup many times
> 2. lookup list many times

--
This message was sent by Atlassian JIRA
(v6.3.4#6332)