address comments

apache · Aug 14, 2024 · df3eb3f · df3eb3f
1 parent ad927fe
commit df3eb3f
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/cpp/velox/substrait/SubstraitToVeloxPlan.cc b/cpp/velox/substrait/SubstraitToVeloxPlan.cc
@@ -1614,8 +1614,10 @@ bool SubstraitToVeloxPlanConverter::canPushdownFunction(
     return false;
   }
 
-  // check whether data type is supported or not
-  if (!veloxTypeList.empty() && fieldIdx < veloxTypeList.size() && !isPushdownSupported(veloxTypeList.at(fieldIdx))) {
+  // Check whether data type is supported or not
+  if (!veloxTypeList.empty() &&
+      fieldIdx < veloxTypeList.size() &&
+      !isPushdownSupported(veloxTypeList.at(fieldIdx))) {
     return false;
   }
 
@@ -1696,7 +1698,7 @@ bool SubstraitToVeloxPlanConverter::canPushdownOr(
 }
 
 bool SubstraitToVeloxPlanConverter::isPushdownSupported(TypePtr inputType) {
-  // keep the same with mapToFilters
+  // Keep the same with mapToFilters
   switch (inputType->kind()) {
     case TypeKind::TIMESTAMP:
     case TypeKind::VARBINARY: