Remove file list empty check to support bucketed scan

WangGuangxin · web-flow · commit 42f71e060cc2 · 2023-07-13T21:37:37.000+08:00
diff --git a/velox/substrait/SubstraitToVeloxPlan.cpp b/velox/substrait/SubstraitToVeloxPlan.cpp
@@ -1195,7 +1195,8 @@ int32_t SubstraitVeloxPlanConverter::streamIsInput(
   if (sRead.has_local_files()) {
     const auto& fileList = sRead.local_files().items();
     if (fileList.size() == 0) {
-      VELOX_FAIL("At least one file path is expected.");
+      // bucketed scan may contains empty file list
+      return -1;
     }
 
     // The stream input will be specified with the format of

Original file line number	Diff line number	Diff line change
`@@ -1195,7 +1195,8 @@ int32_t SubstraitVeloxPlanConverter::streamIsInput(`
`1195`	`1195`	`if (sRead.has_local_files()) {`
`1196`	`1196`	`const auto& fileList = sRead.local_files().items();`
`1197`	`1197`	`if (fileList.size() == 0) {`
`1198`		`- VELOX_FAIL("At least one file path is expected.");`
	`1198`	`+ // bucketed scan may contains empty file list`
	`1199`	`+ return -1;`
`1199`	`1200`	`}`
`1200`	`1201`
`1201`	`1202`	`// The stream input will be specified with the format of`