vikrantmehta123 opened a new pull request, #2862:
URL: https://github.com/apache/datafusion-ballista/pull/2862
# Which issue does this PR close?
Closes #2729.
# Rationale for this change
The issue explains the rationale, as well as the
[comment](https://github.com/apache/datafusion-ballista/pull/2713#discussion_r4427001390).
# What changes are included in this PR?
- Add `with_ballista_execution_mode(ExecutionMode)` and
`ballista_execution_mode()` to `SessionConfigExt`. Unit tests are added
for this.
- Re-export `ExecutionMode` from `ballista::prelude`, so the setter's
argument
type can be named from the client crate.
- Use the new getter at the two scheduler call sites that read the mode
(`state/mod.rs` and `state/task_manager.rs`). Behaviour is unchanged.
- Update the Rust snippet in the pipelined execution user guide to use the
setter.
**Note:** The getter returns `Result<ExecutionMode>` and not the plain
`ExecutionMode` (in the issue, plain ExecutionMode was suggested), as the key
can hold an invalid string when it is set with the SQL `SET` or `set_str`.
Hence, the Result is kept.
# Are there any user-facing changes?
Yes, as this introduces two new methods for users, and there is an
additional export in the prelude.
No existing behavior changes.
--
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]
