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

Ignite TC Bot commented on IGNITE-28986:
----------------------------------------

{panel:title=Branch: [pull/13483/head] Base: [master] : No blockers 
found!|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}{panel}
{panel:title=Branch: [pull/13483/head] Base: [master] : New Tests 
(11)|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}
{color:#00008b}Calcite SQL 3{color} [[tests 
11|https://ci2.ignite.apache.org/viewLog.html?buildId=9286482]]
* {color:#013220}IgniteCalciteTestSuite3: 
LocalQueryIntegrationTest.testLocalQueryWithDifferentDistribution - 
PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = true, partSz = 1] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = true, partSz = 2] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = true, partSz = 5] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = true, partSz = 10] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = true, partSz = 20] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = false, partSz = 1] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = false, partSz = 2] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = false, partSz = 5] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = false, partSz = 10] - PASSED{color}
* {color:#013220}IgniteCalciteTestSuite3: 
QueryWithPartitionsIntegrationTest.testPartitionsQueryWithDifferentDistribution[local
 = false, partSz = 20] - PASSED{color}
... and 0 new tests

{panel}
[TeamCity *--> Run :: All* 
Results|https://ci2.ignite.apache.org/viewLog.html?buildId=9284368&buildTypeId=IgniteTests24Java8_RunAll]
{color:#ffffff}tcbot-analysis-comment chainBuildId=9284368 
rerunBuildIds=9286480,9286482{color}

> Local and Partitions queries are broken if plan contains Exchange
> -----------------------------------------------------------------
>
>                 Key: IGNITE-28986
>                 URL: https://issues.apache.org/jira/browse/IGNITE-28986
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 2.18
>            Reporter: Evgeny Stanilovsky
>            Assignee: Evgeny Stanilovsky
>            Priority: Major
>              Labels: ignite-2, ise
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Local and specific partitions queries [1], [2] are broken if Exchange is 
> raised. 
> How to reproduce is master: 
> For partitions (QueryWithPartitionsIntegrationTest) the same as for local : 
> LocalQueryIntegrationTest#testJoin()
> For exchange emulation i just modify a query : 
> {code:java}
>         String sql = "select /*+ MERGE_JOIN */* from " + table1 + " join " + 
> table2 +
>                         " on " + table1 + "." + joinCol + "=" + table2 + "." 
> + joinCol;
> {code}
> and rel (IgniteMergeJoin) to make exchange relative plan more cheap:
> {code:java}
> IgniteMergeJoin#computeSelfCost -> 
> //add
>         RelNode stripped = getLeft().stripped();
>         boolean inputScan = stripped instanceof TableScan;
>         if (inputScan)
>             return IgniteCost.HUGE;
> {code}
> and here we are :
> plan becomes :
> {code:java}
> IgniteExchange
>   IgniteMergeJoin
>     IgniteExchange
>       IgniteIndexScan
>     IgniteExchange
>       IgniteIndexScan
> {code}
> and tests fails with:
> {code:java}
> java.lang.AssertionError: expected:<3145> but was:<10000>
>       at 
> org.apache.ignite.internal.processors.query.calcite.integration.LocalQueryIntegrationTest.test(LocalQueryIntegrationTest.java:204)
> {code}
> Or (more simple re-producer), just create table with different affinity and 
> try to join with local=true
> It also work for Spool related plans, i.e. plans with spool are also will 
> fail such a tests.
> [1] https://issues.apache.org/jira/browse/IGNITE-19998 SQL Calcite: Add 
> support of setting partitions in SqlFieldsQuery
> [2] https://issues.apache.org/jira/browse/IGNITE-19725 Calcite-2. Add local 
> flag support



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to