Fix hashjoin runtime issue (#106)

zhixingheyi-tian · web-flow · commit 5d4bd100df86 · 2023-01-04T16:50:47.000+08:00
diff --git a/velox/exec/HashBuild.cpp b/velox/exec/HashBuild.cpp
@@ -85,9 +85,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
`@@ -85,9 +85,6 @@ HashBuild::HashBuild(`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`// Identify the non-key build side columns and make a decoder for each.`
`88`		`- const auto numDependents = outputType->size() - numKeys;`
`89`		`- dependentChannels_.reserve(numDependents);`
`90`		`- decoders_.reserve(numDependents);`
`91`	`88`	`for (auto i = 0; i < outputType->size(); ++i) {`
`92`	`89`	`if (keyChannelMap.find(i) == keyChannelMap.end()) {`
`93`	`90`	`dependentChannels_.emplace_back(i);`