diff --git a/velox/exec/fuzzer/WindowFuzzer.cpp b/velox/exec/fuzzer/WindowFuzzer.cpp index a94feb48a6ef..b4d8a578f3a8 100644 --- a/velox/exec/fuzzer/WindowFuzzer.cpp +++ b/velox/exec/fuzzer/WindowFuzzer.cpp @@ -319,17 +319,18 @@ std::string WindowFuzzer::addKRangeOffsetColumnToInputImpl( const SortingKeyAndOrder& orderByKey, const std::string& columnName, const std::string& offsetColumnName) { + // Generate frame bound (constant/column) without nulls. + ScopedVarSetter nullRatioHolder( + &vectorFuzzer_.getMutableOptions().nullRatio, 0.0); + ScopedVarSetter dataSpecHolder( + &vectorFuzzer_.getMutableOptions().dataSpec, {false, false}); + // Use columns as frame bound 50% of time. bool isColumnBound = vectorFuzzer_.coinToss(0.5); const auto type = CppToType::create(); VectorPtr constantFrameBound = isColumnBound ? nullptr : vectorFuzzer_.fuzzConstant(type, 1); VectorPtr columnFrameBound; - // Generate frame bound (constant/column) without nulls. - ScopedVarSetter nullRatioHolder( - &vectorFuzzer_.getMutableOptions().nullRatio, 0.0); - ScopedVarSetter dataSpecHolder( - &vectorFuzzer_.getMutableOptions().dataSpec, {false, false}); velox::test::VectorMaker vectorMaker{pool_.get()}; for (auto i = 0; i < FLAGS_num_batches; i++) {