You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "Paul Rogers (JIRA)" <ji...@apache.org> on 2016/12/19 20:06:58 UTC

[jira] [Created] (DRILL-5135) TestMergeJoinWithSchemaChanges.testNumericTypes doesn't

Paul Rogers created DRILL-5135:
----------------------------------

             Summary: TestMergeJoinWithSchemaChanges.testNumericTypes doesn't
                 Key: DRILL-5135
                 URL: https://issues.apache.org/jira/browse/DRILL-5135
             Project: Apache Drill
          Issue Type: Bug
    Affects Versions: 1.8.0
            Reporter: Paul Rogers
            Assignee: Paul Rogers
            Priority: Minor


The test {{TestMergeJoinWithSchemaChanges.testNumericTypes}} claims to test the merge join. However, the test picks up default session options from its base class, {{BaseTestQuery}}, which leaves the hash join enabled. If one inserts code to dump the physical plan for this test, one finds that the test actually uses a hash join, not a merge join.

The test must be modified to disable hash join. Using the newer test framework:

{code}
    FixtureBuilder builder = ClusterFixture.builder()
        .sessionOption("planner.enable_hashjoin", false)
        .sessionOption("planner.enable_hashagg", false);
{code}

Which produces the following physical plan:

{code}
    "pop" : "merge-join",
{code}




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