fixed variable name

substrait-io · Sep 12, 2024 · e2f54bf · e2f54bf
1 parent b30c8b8
commit e2f54bf
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/src/substrait/textplan/parser/SubstraitPlanRelationVisitor.cpp b/src/substrait/textplan/parser/SubstraitPlanRelationVisitor.cpp
@@ -824,9 +824,9 @@ std::any SubstraitPlanRelationVisitor::visitRelationEmit(
  SymbolType::kRelation);
  auto parentRelationData =
  ANY_CAST(std::shared_ptr<RelationData>, parentSymbol->blob);
- this->processingEmit = true;
+ this->processingEmit_ = true;
  auto result = visitChildren(ctx);
- this->processingEmit = false;
+ this->processingEmit_ = false;
  auto parentRelationType = ANY_CAST(RelationType, parentSymbol->subtype);
  auto common =
  findCommonRelation(parentRelationType, &parentRelationData->relation);
@@ -2030,7 +2030,7 @@ std::pair<int, int> SubstraitPlanRelationVisitor::findFieldReferenceByName(
  std::shared_ptr<RelationData>& relationData,
  const std::string& name) {
  auto fieldReferencesSize = relationData->fieldReferences.size();
- if (isAggregate(symbol) && this->processingEmit) {
+ if (isAggregate(symbol) && this->processingEmit_) {
  fieldReferencesSize = 0;
  }
 

diff --git a/src/substrait/textplan/parser/SubstraitPlanRelationVisitor.h b/src/substrait/textplan/parser/SubstraitPlanRelationVisitor.h
@@ -205,7 +205,7 @@ class SubstraitPlanRelationVisitor : public SubstraitPlanTypeVisitor {
  bool hasSubquery(SubstraitPlanParser::ExpressionContext* ctx);
 
  const SymbolInfo* currentRelationScope_{nullptr}; // Not owned.
- bool processingEmit{false};
+ bool processingEmit_{false};
 };
 
 } // namespace io::substrait::textplan
diff --git a/src/substrait/textplan/parser/SubstraitPlanSubqueryRelationVisitor.cpp b/src/substrait/textplan/parser/SubstraitPlanSubqueryRelationVisitor.cpp
@@ -874,9 +874,9 @@ std::any SubstraitPlanSubqueryRelationVisitor::visitRelationEmit(
  SymbolType::kRelation);
  auto parentRelationData =
  ANY_CAST(std::shared_ptr<RelationData>, parentSymbol->blob);
- this->processingEmit = true;
+ this->processingEmit_ = true;
  auto result = visitChildren(ctx);
- this->processingEmit = false;
+ this->processingEmit_ = false;
  auto parentRelationType = ANY_CAST(RelationType, parentSymbol->subtype);
  auto common =
  findCommonRelation(parentRelationType, &parentRelationData->relation);
@@ -2168,7 +2168,7 @@ SubstraitPlanSubqueryRelationVisitor::findFieldReferenceByName(
  std::shared_ptr<RelationData>& relationData,
  const std::string& name) {
  auto fieldReferencesSize = relationData->fieldReferences.size();
- if (isAggregate(symbol) && this->processingEmit) {
+ if (isAggregate(symbol) && this->processingEmit_) {
  fieldReferencesSize = 0;
  }
 

diff --git a/src/substrait/textplan/parser/SubstraitPlanSubqueryRelationVisitor.h b/src/substrait/textplan/parser/SubstraitPlanSubqueryRelationVisitor.h
@@ -217,7 +217,7 @@ class SubstraitPlanSubqueryRelationVisitor : public SubstraitPlanTypeVisitor {
  bool isWithinSubquery(SubstraitPlanParser::RelationContext* ctx);
 
  const SymbolInfo* currentRelationScope_{nullptr}; // Not owned.
- bool processingEmit{false};
+ bool processingEmit_{false};
 };
 
 } // namespace io::substrait::textplan