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

Add hook to override the DefaultOptimizerList

    Details

      Description

      The optimizer list is a LinkedList that implements the IASTOptimizer interface. This list is currently constructed for query / update evaluated in AST2BOpContext<init>:

              this.optimizers = new DefaultOptimizerList(db.isBottomUpEvaluation());
      

      This ticket is to provide a hook to override the optimizer list.

        Activity

        Hide
        bryanthompson bryanthompson added a comment -

        Committed fix to SF GIT branch TICKET_1113 af9bef109b6b5aa1f470ac2aebdbe33dccc968c4

        This commit defines a new system property "ASTOptimizerClass". When defined, this property must name a class that extends ASTOptimizerList. An instance of that class is then constructed and used to optimize the queries.s

        I refactored the AbstractTripleStore.isBottomUpEvaluation() support to then walk the returned list and remove the ASTBottomUpOptimizer when bottom-up evaluation is not supported. This should preserve the semantics of that option while making the constructor for the ASTOptimizerClass stateless.

        Show
        bryanthompson bryanthompson added a comment - Committed fix to SF GIT branch TICKET_1113 af9bef109b6b5aa1f470ac2aebdbe33dccc968c4 This commit defines a new system property "ASTOptimizerClass". When defined, this property must name a class that extends ASTOptimizerList. An instance of that class is then constructed and used to optimize the queries.s I refactored the AbstractTripleStore.isBottomUpEvaluation() support to then walk the returned list and remove the ASTBottomUpOptimizer when bottom-up evaluation is not supported. This should preserve the semantics of that option while making the constructor for the ASTOptimizerClass stateless.
        Hide
        bryanthompson bryanthompson added a comment -

        Assigned to brad to review and merge.

        Show
        bryanthompson bryanthompson added a comment - Assigned to brad to review and merge.
        Hide
        bradbebee bradbebee added a comment -

        Branch merged in SF and GH. Stage for 1.5.1 release.

        Show
        bradbebee bradbebee added a comment - Branch merged in SF and GH. Stage for 1.5.1 release.

          People

          • Assignee:
            beebs Brad Bebee
            Reporter:
            bryanthompson bryanthompson
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: