aglinxinyuan commented on PR #5720: URL: https://github.com/apache/texera/pull/5720#issuecomment-4704595462
Addressed both points in `01c8603`: 1. **Moved the flag to `PhysicalOp`** (it's a physical-execution property) + a `withRequiresMaterializedExecution` builder; removed it from `LogicalOp`. 2. **Real consumer in the scheduler**: `CostBasedScheduleGenerator` now forces a fully-materialized schedule when any physical op requires it (otherwise the existing PIPELINED/MATERIALIZED logic runs unchanged). Dropped the submission-time `WorkflowExecutionService.validateExecutionMode`. Default `false`, so it's dormant and behavior-preserving today; the loop operators set it on their `PhysicalOp`. PR is now 3 files: `PhysicalOp`, `CostBasedScheduleGenerator`, `WorkflowCoreTypesSpec`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
