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.

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

Fix parse error for query_events (fix for PR#360) #397

Conversation

dongwook-chan
Copy link
Collaborator

related issue: python-mysql-replication issue#380, pg_cameleon commit#38c7a90

Analysis

  • Could not parse status variables in query events for MariaDB
  • Because the library was not aware of MariaDB-exclusive status variables

Implementation

  • Added MariaDB-only status variables
  • Added Exception to raise when status variable parse failed
  • Added tests for parsing query events for MariaDB

dongwook-chan referenced this pull request in the4thdoctor/pg_chameleon Feb 25, 2023
…#38c7a90

Analysis
- Could not parse status variables in query events for MariaDB
- Because the library was not aware of MariaDB-exclusive status variables

Implementation
- Added MariaDB-only status variables
- Added Exception to raise when status variable parse failed
- Added tests for parsing query events for MariaDB
@the4thdoctor
Copy link

Any chance to get this PR merged and a new release out?
I'd like to make a new release for pgchameleon as well :) .
ta!

@dongwook-chan
Copy link
Collaborator Author

@julien-duponchelle
Just in case you haven't noticed this PR.
I appreciate your effort to keep this project going!

@julien-duponchelle
Copy link
Owner

Thanks I'm trying to get it merge and release during the week end

@julien-duponchelle julien-duponchelle merged commit 3957b7e into julien-duponchelle:main Mar 12, 2023
@julien-duponchelle
Copy link
Owner

Released!

@the4thdoctor
Copy link

ta!
I'll make a new release for pgchameleon in the next couple of days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants