diff --git a/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc b/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc index c18d265986f8..7bb0eab77758 100644 --- a/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc +++ b/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc @@ -68,7 +68,8 @@ static const std::unordered_set kBlackList = { "trunc", "sequence", "approx_percentile", - "get_array_struct_fields"}; + "get_array_struct_fields", + "map_from_arrays"}; } // namespace @@ -189,9 +190,6 @@ bool SubstraitToVeloxPlanValidator::validateScalarFunction( return validateRound(scalarFunction, inputType); } else if (name == "extract") { return validateExtractExpr(params); - } else if (name == "map_from_arrays") { - LOG_VALIDATION_MSG("map_from_arrays is not supported."); - return false; } else if (name == "concat") { for (const auto& type : types) { if (type.find("struct") != std::string::npos || type.find("map") != std::string::npos || @@ -200,14 +198,6 @@ bool SubstraitToVeloxPlanValidator::validateScalarFunction( return false; } } - } else if (name == "murmur3hash") { - for (const auto& type : types) { - if (type.find("struct") != std::string::npos || type.find("map") != std::string::npos || - type.find("list") != std::string::npos) { - LOG_VALIDATION_MSG(type + " is not supported in murmur3hash."); - return false; - } - } } // Validate regex functions.