alamb commented on issue #3033:
URL:
https://github.com/apache/arrow-datafusion/issues/3033#issuecomment-1205085924
As I understand it, you are trying to identify all metrics for a particular
query execution within a session
The `session_id` on `SessionState` already identifies the session, but a
single session can run multiple queries
What about adding a `statement_id` to the `SessionState`? Like:
```rust
/// Execution context for registering data sources and executing queries
#[derive(Clone)]
pub struct SessionState {
/// Uuid for the session
pub session_id: String,
/// Statement id -- each query / statement increases this number. <-----
this is NEW
pub statement_id: usize,
/// Responsible for optimizing a logical plan
pub optimizer: Optimizer,
... all the same ...
}
```
Then you could use the combination of `(session_id, statement_id)` to
connect up metrics across nodes?
--
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]
