Skip to content

Commit addc9d3

Browse files
rui-mozhejiangxiaomai
authored andcommitted
Fallback murmur3hash on complex types (#299)
1 parent eea7ece commit addc9d3

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

velox/substrait/SubstraitToVeloxPlanValidator.cpp

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,16 @@ bool SubstraitToVeloxPlanValidator::validateScalarFunction(
146146
return false;
147147
}
148148
}
149+
if (name == "murmur3hash") {
150+
for (const auto& type : types) {
151+
if (type.find("struct") != std::string::npos ||
152+
type.find("map") != std::string::npos ||
153+
type.find("list") != std::string::npos) {
154+
VLOG(1) << type << "is not supported in murmur3hash.";
155+
return false;
156+
}
157+
}
158+
}
149159

150160
std::unordered_set<std::string> functions = {
151161
"regexp_replace", "split", "split_part",

0 commit comments

Comments
 (0)