Skip to content

Commit

Permalink
Fix compile
Browse files Browse the repository at this point in the history
  • Loading branch information
yixinglu committed Dec 3, 2022
1 parent 90e497b commit 32e9e65
Show file tree
Hide file tree
Showing 7 changed files with 15 additions and 15 deletions.
6 changes: 3 additions & 3 deletions src/graph/executor/query/InnerJoinExecutor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -246,15 +246,15 @@ void InnerJoinExecutor::buildNewRow(const std::unordered_map<T, std::vector<cons

const std::string& InnerJoinExecutor::leftVar() const {
if (node_->kind() == PlanNode::Kind::kHashInnerJoin) {
return node_->asNode<BiJoin>()->leftInputVar();
return node_->asNode<HashJoin>()->leftInputVar();
} else {
return node_->asNode<Join>()->leftVar().first;
}
}

const std::string& InnerJoinExecutor::rightVar() const {
if (node_->kind() == PlanNode::Kind::kHashInnerJoin) {
return node_->asNode<BiJoin>()->rightInputVar();
return node_->asNode<HashJoin>()->rightInputVar();
} else {
return node_->asNode<Join>()->rightVar().first;
}
Expand All @@ -267,7 +267,7 @@ HashInnerJoinExecutor::HashInnerJoinExecutor(const PlanNode* node, QueryContext*

folly::Future<Status> HashInnerJoinExecutor::execute() {
SCOPED_TIMER(&execTime_);
auto* joinNode = asNode<BiJoin>(node());
auto* joinNode = asNode<HashJoin>(node());
NG_RETURN_IF_ERROR(checkBiInputDataSets());
return join(joinNode->hashKeys(), joinNode->probeKeys(), joinNode->colNames());
}
Expand Down
2 changes: 1 addition & 1 deletion src/graph/executor/query/JoinExecutor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ Status JoinExecutor::checkInputDataSets() {
}

Status JoinExecutor::checkBiInputDataSets() {
auto* join = asNode<BiJoin>(node());
auto* join = asNode<HashJoin>(node());
lhsIter_ = ectx_->getResult(join->leftInputVar()).iter();
DCHECK(!!lhsIter_);
if (lhsIter_->isGetNeighborsIter() || lhsIter_->isDefaultIter()) {
Expand Down
2 changes: 1 addition & 1 deletion src/graph/executor/query/LeftJoinExecutor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,7 @@ HashLeftJoinExecutor::HashLeftJoinExecutor(const PlanNode* node, QueryContext* q

folly::Future<Status> HashLeftJoinExecutor::execute() {
SCOPED_TIMER(&execTime_);
auto* joinNode = asNode<BiJoin>(node());
auto* joinNode = asNode<HashJoin>(node());
auto& rhsResult = ectx_->getResult(joinNode->rightInputVar());
rightColSize_ = rhsResult.valuePtr()->getDataSet().colNames.size();
NG_RETURN_IF_ERROR(checkBiInputDataSets());
Expand Down
2 changes: 1 addition & 1 deletion src/graph/planner/match/ShortestPathPlanner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ namespace graph {
// +------------+---------------+
// |
// +--------+---------+
// |CrossJoin|
// | CrossJoin |
// +--------+---------+
// |
// +--------+---------+
Expand Down
2 changes: 1 addition & 1 deletion src/graph/visitor/PrunePropertiesVisitor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -427,7 +427,7 @@ void PrunePropertiesVisitor::pruneCurrent(AppendVertices *node) {
node->setVertexProps(std::move(prunedVertexProps));
}

void PrunePropertiesVisitor::visit(BiJoin *node) {
void PrunePropertiesVisitor::visit(HashJoin *node) {
status_ = depsPruneProperties(node->dependencies());
}

Expand Down
2 changes: 1 addition & 1 deletion src/graph/visitor/PrunePropertiesVisitor.h
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ class PrunePropertiesVisitor final : public PlanNodeVisitor {
// prune properties in AppendVertices according to the used properties collected previous
void pruneCurrent(AppendVertices *node);

void visit(BiJoin *node) override;
void visit(HashJoin *node) override;

void visit(Union *node) override;
void visit(CrossJoin *node) override;
Expand Down
14 changes: 7 additions & 7 deletions tests/tck/features/optimizer/PrunePropertiesRule.feature

Large diffs are not rendered by default.

0 comments on commit 32e9e65

Please sign in to comment.