QuakeWang opened a new pull request, #2215: URL: https://github.com/apache/datafusion-ballista/pull/2215
# Which issue does this PR close? Closes #2214 # Rationale for this change `StaticExecutionGraph` removed a stage before running fallible plan transformations. If a transformation failed, the error returned before the stage was reinserted, leaving the execution graph without the original stage. # What changes are included in this PR? Build replacement stages before updating the stage map, so failed resolve and rollback transitions preserve the original stage. Add regression tests for all three affected transitions. # Are there any user-facing changes? No API changes. Failed stage transitions now preserve execution graph state. # Testing - `ci/scripts/rust_fmt.sh` - Scheduler all-targets/all-features Clippy - Scheduler library tests -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
