Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove quotes from keyspace names before sending them back in set_keyspace result messages #129

Merged
merged 6 commits into from
Jul 30, 2024
Merged

Remove quotes from keyspace names before sending them back in set_keyspace result messages #129

merged 6 commits into from
Jul 30, 2024

Commits on Jul 29, 2024

  1. Drive-by fix while working on this issue. Noted there wasn't really a… 

    … clear way to tell from

logs whether a given query was handled by the proxy or passed on to upstream servers.  Added
some minimal logging to provide some additional info on this point.
    @absurdfarce @lukasz-antoniak
    absurdfarce authored and lukasz-antoniak committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    54ceb7e View commit details
    Browse the repository at this point in the history

  2. Added some explanatory comments

    @absurdfarce @lukasz-antoniak
    absurdfarce authored and lukasz-antoniak committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    5813a9f View commit details
    Browse the repository at this point in the history

  3. Adding test per code review comments

    @absurdfarce @lukasz-antoniak
    absurdfarce authored and lukasz-antoniak committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    dbdd986 View commit details
    Browse the repository at this point in the history

  4. Clean up logging

    @absurdfarce @lukasz-antoniak
    absurdfarce authored and lukasz-antoniak committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    bd60d74 View commit details
    Browse the repository at this point in the history

  5. Return detailed C* error for USE query. More test comments

    @lukasz-antoniak
    lukasz-antoniak committed Jul 29, 2024
    Configuration menu
    Copy the full SHA
    cd418fe View commit details
    Browse the repository at this point in the history

Commits on Jul 30, 2024

  1. Review comments

    @lukasz-antoniak
    lukasz-antoniak committed Jul 30, 2024
    Configuration menu
    Copy the full SHA
    68126a8 View commit details
    Browse the repository at this point in the history