-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Cassandra session level tracking #620
Comments
Claude-at-Instaclustr
changed the title
Add Cassandra sessions level tracking
Add Cassandra session level tracking
Apr 26, 2022
we ended up taking a somewhat different approach but the proper handling of use statements was implemented by #806 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
PR #598 (Use cql3 parser) brings with it the ability to process non-SQL commands.
One of the commands
USE <keyspace>
changes the state of the connection on the server side. In order for Shotover to properly process commands after theUSE
command it must track the keyspace named in the command and apply it to parsed names where the keyspace is permitted but not provided. Effectively, theUSE
command sets a default keyspace.This change should figure out how to track sessions (not requests) between the client and the server so that the default namespace, and similar values, can be tracked across the requests. The session ID may be different from the stream ID found in the Cassandra frame itself. Part of this issue is to determine if that is the case or not.
The end result is that we will have the ability to store and clear session data so that data that crosses requests can be handled efficiently and consistently across the code base.
The text was updated successfully, but these errors were encountered: