Uploaded image for project: 'Blazegraph (by SYSTAP)'
  1. Blazegraph (by SYSTAP)
  2. BLZG-1020

StaticOptimizer should always choose a ZERO cardinality tail first

    Details

      Description

      If there is a tail in the join group with a cardinality of zero, this should always run first regardless of what else is going on with the ancestry or otherwise.

        Activity

        beebs Brad Bebee created issue -
        Hide
        bryanthompson bryanthompson added a comment -

        Mike has fixed this in the 1.3.x maintenance and development branch in r8387.

        We have back-ported this fix to the maintenance branch for 1.2.4 and 1.2.5.

        Bug fix to BLZG-1020 (StaticOptimizer should always choose a ZERO cardinality tail first) for the 1.2.5 tag. This commit is against branches/BIGDATA_RELEASE_1_2_4. The 1.2.5 release was made from this branch. A 1.2.6 release will also be made from this branch.

                    /**
                     * Always choose a ZERO cardinality tail first, regardless of
                     * ancestry.
                     * 
                     * @see <a href="http://trac.bigdata.com/ticket/955" >
                     *      StaticOptimizer should always choose a ZERO cardinality tail
                     *      first </a>
                     */
                    for (int i = 0; i < arity; i++) {
                        if (cardinality(i) == 0) {
                            preferredFirstTail = i;
                            break;
                        }
                    }
        

        I will enroll this branch in CI.

        Show
        bryanthompson bryanthompson added a comment - Mike has fixed this in the 1.3.x maintenance and development branch in r8387. We have back-ported this fix to the maintenance branch for 1.2.4 and 1.2.5. Bug fix to BLZG-1020 (StaticOptimizer should always choose a ZERO cardinality tail first) for the 1.2.5 tag. This commit is against branches/BIGDATA_RELEASE_1_2_4. The 1.2.5 release was made from this branch. A 1.2.6 release will also be made from this branch. /** * Always choose a ZERO cardinality tail first, regardless of * ancestry. * * @see <a href="http://trac.bigdata.com/ticket/955" > * StaticOptimizer should always choose a ZERO cardinality tail * first </a> */ for (int i = 0; i < arity; i++) { if (cardinality(i) == 0) { preferredFirstTail = i; break; } } I will enroll this branch in CI.
        beebs Brad Bebee made changes -
        Field Original Value New Value
        Workflow Trac Import v2 [ 12856 ] Trac Import v3 [ 14348 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v3 [ 14348 ] Trac Import v4 [ 15677 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v4 [ 15677 ] Trac Import v5 [ 17063 ]
        beebs Brad Bebee made changes -
        Labels Issue_patch_20150625
        beebs Brad Bebee made changes -
        Status Closed - Won't Fix [ 6 ] Open [ 1 ]
        beebs Brad Bebee made changes -
        Status Open [ 1 ] Accepted [ 10101 ]
        beebs Brad Bebee made changes -
        Status Accepted [ 10101 ] In Progress [ 3 ]
        beebs Brad Bebee made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        beebs Brad Bebee made changes -
        Status Resolved [ 5 ] In Review [ 10100 ]
        beebs Brad Bebee made changes -
        Resolution Fixed [ 1 ] Done [ 10000 ]
        Status In Review [ 10100 ] Done [ 10000 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v5 [ 17063 ] Trac Import v6 [ 18245 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v6 [ 18245 ] Trac Import v7 [ 19642 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v7 [ 19642 ] Trac Import v8 [ 21264 ]

          People

          • Assignee:
            mikepersonick mikepersonick
            Reporter:
            mikepersonick mikepersonick
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: