[native] Remove const from VeloxPlanValidator.

prestodb · Dec 4, 2024 · 72b2324 · 72b2324
1 parent a524d33
commit 72b2324
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/presto-native-execution/presto_cpp/main/types/VeloxPlanConversion.cpp b/presto-native-execution/presto_cpp/main/types/VeloxPlanConversion.cpp
@@ -36,7 +36,7 @@ namespace facebook::presto {
 protocol::PlanConversionResponse prestoToVeloxPlanConversion(
     const std::string& planFragmentJson,
     memory::MemoryPool* pool,
-    const VeloxPlanValidator* planValidator) {
+    VeloxPlanValidator* planValidator) {
   protocol::PlanConversionResponse response;
 
   try {

diff --git a/presto-native-execution/presto_cpp/main/types/VeloxPlanConversion.h b/presto-native-execution/presto_cpp/main/types/VeloxPlanConversion.h
@@ -27,6 +27,6 @@ namespace facebook::presto {
 protocol::PlanConversionResponse prestoToVeloxPlanConversion(
     const std::string& planFragmentJson,
     velox::memory::MemoryPool* pool,
-    const VeloxPlanValidator* planValidator);
+    VeloxPlanValidator* planValidator);
 
 } // namespace facebook::presto
diff --git a/presto-native-execution/presto_cpp/main/types/VeloxPlanValidator.cpp b/presto-native-execution/presto_cpp/main/types/VeloxPlanValidator.cpp
@@ -33,7 +33,7 @@ bool planHasNestedJoinLoop(const velox::core::PlanNodePtr planNode) {
 }
 
 void VeloxPlanValidator::validatePlanFragment(
-    const velox::core::PlanFragment& fragment) const {
+    const velox::core::PlanFragment& fragment) {
   const auto failOnNestedLoopJoin =
       SystemConfig::instance()
           ->optionalProperty<bool>(

diff --git a/presto-native-execution/presto_cpp/main/types/VeloxPlanValidator.h b/presto-native-execution/presto_cpp/main/types/VeloxPlanValidator.h
@@ -17,8 +17,7 @@
 namespace facebook::presto {
 class VeloxPlanValidator {
  public:
-  virtual void validatePlanFragment(
-      const velox::core::PlanFragment& fragment) const;
+  virtual void validatePlanFragment(const velox::core::PlanFragment& fragment);
   virtual ~VeloxPlanValidator() = default;
 };
 } // namespace facebook::presto