You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@quickstep.apache.org by zu...@apache.org on 2017/01/06 21:46:07 UTC

incubator-quickstep git commit: Fixed the bug in resetting exactness for InsertSelect.

Repository: incubator-quickstep
Updated Branches:
  refs/heads/fix-stats [created] c0d510eb0


Fixed the bug in resetting exactness for InsertSelect.


Project: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/commit/c0d510eb
Tree: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/tree/c0d510eb
Diff: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/diff/c0d510eb

Branch: refs/heads/fix-stats
Commit: c0d510eb0469c095cc216846d2da5df65961507b
Parents: 9fcb0ac
Author: Zuyu Zhang <zu...@apache.org>
Authored: Fri Jan 6 13:45:59 2017 -0800
Committer: Zuyu Zhang <zu...@apache.org>
Committed: Fri Jan 6 13:45:59 2017 -0800

----------------------------------------------------------------------
 query_optimizer/ExecutionGenerator.cpp      | 2 +-
 relational_operators/SaveBlocksOperator.hpp | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-quickstep/blob/c0d510eb/query_optimizer/ExecutionGenerator.cpp
----------------------------------------------------------------------
diff --git a/query_optimizer/ExecutionGenerator.cpp b/query_optimizer/ExecutionGenerator.cpp
index d24f498..29e67f7 100644
--- a/query_optimizer/ExecutionGenerator.cpp
+++ b/query_optimizer/ExecutionGenerator.cpp
@@ -1243,7 +1243,7 @@ void ExecutionGenerator::convertInsertSelection(
   insert_destination_proto->set_relational_op_index(insert_selection_index);
 
   CatalogRelation *mutable_relation =
-      catalog_database_->getRelationByIdMutable(selection_relation->getID());
+      catalog_database_->getRelationByIdMutable(destination_relation.getID());
   const QueryPlan::DAGNodeIndex save_blocks_index =
       execution_plan_->addRelationalOperator(
           new SaveBlocksOperator(query_handle_->query_id(), mutable_relation));

http://git-wip-us.apache.org/repos/asf/incubator-quickstep/blob/c0d510eb/relational_operators/SaveBlocksOperator.hpp
----------------------------------------------------------------------
diff --git a/relational_operators/SaveBlocksOperator.hpp b/relational_operators/SaveBlocksOperator.hpp
index a8d5327..573d81e 100644
--- a/relational_operators/SaveBlocksOperator.hpp
+++ b/relational_operators/SaveBlocksOperator.hpp
@@ -61,9 +61,9 @@ class SaveBlocksOperator : public RelationalOperator {
    * @param force If true, force writing of all blocks to disk, otherwise only
    *        write dirty blocks.
    **/
-  explicit SaveBlocksOperator(const std::size_t query_id,
-                              CatalogRelation *relation,
-                              const bool force = false)
+  SaveBlocksOperator(const std::size_t query_id,
+                     CatalogRelation *relation,
+                     const bool force = false)
       : RelationalOperator(query_id),
         force_(force),
         relation_(relation),