Github user zuyu commented on a diff in the pull request:

    https://github.com/apache/incubator-quickstep/pull/300#discussion_r140570548
  
    --- Diff: query_optimizer/PhysicalGenerator.cpp ---
    @@ -175,6 +176,8 @@ P::PhysicalPtr PhysicalGenerator::optimizePlan() {
         rules.push_back(std::make_unique<Partition>(optimizer_context_));
       }
     
    +  rules.emplace_back(new FuseHashSelect());
    --- End diff --
    
    You may want to add a `gflags` with a default `true` value to switch 
between using this rule or not, w/o rebuilding the binary.


---

Reply via email to