Skip to content

Commit

Permalink
Minor: improve docstrings on SessionState (#7654)
Browse files Browse the repository at this point in the history
* Minor: improve docstrings on `SessionState`

* Update datafusion/core/src/execution/context.rs

Co-authored-by: Liang-Chi Hsieh <[email protected]>

---------

Co-authored-by: Liang-Chi Hsieh <[email protected]>
  • Loading branch information
alamb and viirya authored Sep 25, 2023
1 parent 2d11436 commit 3505b7f
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion datafusion/core/src/execution/context.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1348,7 +1348,12 @@ impl QueryPlanner for DefaultQueryPlanner {
}
}

/// Execution context for registering data sources and executing queries
/// Execution context for registering data sources and executing queries.
/// See [`SessionContext`] for a higher level API.
///
/// Note that there is no `Default` or `new()` for SessionState,
/// to avoid accidentally running queries or other operations without passing through
/// the [`SessionConfig`] or [`RuntimeEnv`]. See [`SessionContext`].
#[derive(Clone)]
pub struct SessionState {
/// A unique UUID that identifies the session
Expand Down

0 comments on commit 3505b7f

Please sign in to comment.