Andrew Lamb

> @alamb Hello! Have you had already time to check the PR? Hi @gandronchik sadly I have not had a chance. I apologize for my lack of bandwidth but it...

Thank you @avantgardnerio

Wow -- that almost is worth a blog post itself!

Seems reasonable to me

sqlparser support was added here: https://github.com/sqlparser-rs/sqlparser-rs/pull/569 I'll try and get a new release of that library out in a day or two

I think the idea of pinning the `Cargo.lock` for datafusion-cli was from https://github.com/apache/arrow-datafusion/issues/2071 / https://github.com/apache/arrow-datafusion/pull/2112 and @happysalada I don't have a strong preference to be honest, as I normally run...

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...

> A single session could have multiple statements run concurrently. And a statement_id is not enough to differ metrics either. That is a good point. I still think `statement_id` is...

I plan to review this carefully today -- thanks @ovr

Thanks -- I'll try and find some more time to review this PR. On Wed, May 18, 2022 at 5:54 PM Dmitry Patsura ***@***.***> wrote: > ***@***.**** commented on this...