You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafodion.apache.org by "Hans Zeller (JIRA)" <ji...@apache.org> on 2017/05/17 02:33:04 UTC

[jira] [Updated] (TRAFODION-2616) Nested join regression after fix for TRAFODION-2569

     [ https://issues.apache.org/jira/browse/TRAFODION-2616?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Hans Zeller updated TRAFODION-2616:
-----------------------------------
    Summary: Nested join regression after fix for TRAFODION-2569  (was: Nested join regression on salted tables after fix for TRAFODION-2569)

> Nested join regression after fix for TRAFODION-2569
> ---------------------------------------------------
>
>                 Key: TRAFODION-2616
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-2616
>             Project: Apache Trafodion
>          Issue Type: Bug
>          Components: sql-cmp
>    Affects Versions: 2.2-incubating
>            Reporter: Hans Zeller
>            Assignee: Hans Zeller
>             Fix For: 2.2-incubating
>
>
> The fix for TRAFODION-2569 caused a regression with nested joins on salted tables: If the salted table ends up as the inner table of a join, we omit computed predicates on the salt column.
> The problem happened when I removed the call to removeIndexInfo in file memo.cpp. The FilterRule0 code called Scan::copyIndexInfo(), which is not correct when pushing predicates into a scan node, because additional indexes could become interesting.
> The "kludge" in memo.cpp compensated for this by calling removeIndexInfo(), then calling addIndexInfo(). A better way is probably to call addIndexInfo() in the first place in the filter rule.
> Also, method copyIndexInfo should copy the computed predicates, since the only place where they are computed is in addIndexInfo().
> I probably still won't be able to remove the "kludge" in memo.cpp.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)