search

deephaven / deephaven-core

Deephaven Community Core
Other
256 stars 80 forks source link

Implement Flight CloseSession #6284

Open devinrsmith opened 3 weeks ago

devinrsmith commented 3 weeks ago

Flight has recently added some actions around session management; in particular, closing a session. https://github.com/apache/arrow/issues/34865

We should investigate if this would be a reasonable replacement for our own SessionService CloseSession.

Specifically, this would allow FlightSQL clients to explicitly close their session, which makes closing of the resources opened during the FlightSQL session much easier than a simple timeout.

This is likely dependent on #5339 (which adds ActionResolver interface) and upgrading Arrow to 16+.

devinrsmith commented 3 weeks ago

We will need to figure out how features like this will need to interact with existing authorization hooks like io.deephaven.auth.codegen.impl.SessionServiceAuthWiring.