Fix hashjoin runtime issue (#106)

zhixingheyi-tian · zhejiangxiaomai · commit 91ddb55afbee · 2023-02-22T11:17:45.000+08:00
diff --git a/velox/exec/HashBuild.cpp b/velox/exec/HashBuild.cpp
@@ -86,9 +86,6 @@ HashBuild::HashBuild(
   }
 
   // Identify the non-key build side columns and make a decoder for each.
-  const auto numDependents = outputType->size() - numKeys;
-  dependentChannels_.reserve(numDependents);
-  decoders_.reserve(numDependents);
   for (auto i = 0; i < outputType->size(); ++i) {
     if (keyChannelMap.find(i) == keyChannelMap.end()) {
       dependentChannels_.emplace_back(i);

Original file line number	Diff line number	Diff line change
`@@ -86,9 +86,6 @@ HashBuild::HashBuild(`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`// Identify the non-key build side columns and make a decoder for each.`
`89`		`- const auto numDependents = outputType->size() - numKeys;`
`90`		`- dependentChannels_.reserve(numDependents);`
`91`		`- decoders_.reserve(numDependents);`
`92`	`89`	`for (auto i = 0; i < outputType->size(); ++i) {`
`93`	`90`	`if (keyChannelMap.find(i) == keyChannelMap.end()) {`
`94`	`91`	`dependentChannels_.emplace_back(i);`