From 96c51ae1b89c99efeb6d7fe3eab10ca00e15124a Mon Sep 17 00:00:00 2001
From: Pedro Eugenio Rocha Pedreira <pedroerp@meta.com>
Date: Wed, 22 May 2024 17:08:50 -0700
Subject: [PATCH] Re-enable alternative plans in join fuzzer (#9898)

Summary:
Pull Request resolved: https://github.com/facebookincubator/velox/pull/9898

Alternative join plans (including sort + merge join) were accidentaly
disabled in a recent refactor
(https://github.com/facebookincubator/velox/pull/9045).
Re-enabling them in this PR.

Reviewed By: mbasmanova

Differential Revision: D57696398

fbshipit-source-id: d0264a1fa214f0e1b46ab03b54fa76fd5fb59173
---
 velox/exec/fuzzer/JoinFuzzer.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/velox/exec/fuzzer/JoinFuzzer.cpp b/velox/exec/fuzzer/JoinFuzzer.cpp
index 3d828eaaed63..88caa5fb236a 100644
--- a/velox/exec/fuzzer/JoinFuzzer.cpp
+++ b/velox/exec/fuzzer/JoinFuzzer.cpp
@@ -980,6 +980,10 @@ void JoinFuzzer::verify(core::JoinType joinType) {
       flatBuildInput,
       outputColumns));
 
+  makeAlternativePlans(defaultPlan.plan, probeInput, buildInput, altPlans);
+  makeAlternativePlans(
+      defaultPlan.plan, flatProbeInput, flatBuildInput, altPlans);
+
   const auto tableScanDir = exec::test::TempDirectoryPath::create();
   addPlansWithTableScan(
       tableScanDir->getPath(),