Skip to content

Conversation

@elianddb
Copy link

@elianddb elianddb commented Oct 27, 2025

Fixes dolthub/dolt#9887

  • Add TypeName() to binlogEvent objects for error message creation on the frontend, i.e. go-mysql-server.
  • Add ERBase64DecodeError = 1575, ERNoFormatDescriptionEventBeforeBinlogStatement = 1609, and EROnlyFDAndRBREventsAllowedInBinlogStatement = 1730 for Binlog statements error handling.
  • Add Binlog statement parser support.
  • Add mariadb executable comment support and fix handling of empty comments, i.e. /*!*/ and /*M!*/.

@elianddb elianddb requested a review from zachmu as a code owner October 27, 2025 21:21
@elianddb elianddb changed the title dolthub/dolt#9887: Fix executable comments and add support for BINLOG and mariadb executable comments dolthub/dolt#9887: Fix empty executable comments and add BINLOG and mariadb executable comments Oct 28, 2025
@elianddb elianddb changed the title dolthub/dolt#9887: Fix empty executable comments and add BINLOG and mariadb executable comments dolthub/dolt#9887: Fix empty executable comments and add BINLOG support and mariadb executable comments Oct 28, 2025
@elianddb elianddb requested a review from fulghum October 29, 2025 17:05
@elianddb elianddb force-pushed the elian/9887 branch 2 times, most recently from 7a4665b to 02e4ea1 Compare October 30, 2025 23:12
Copy link

@fulghum fulghum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@elianddb elianddb merged commit b08b393 into main Nov 5, 2025
4 checks passed
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.

Support MariaDB mariadb-binlog which translates a MariaDB binlog to SQL queries

3 participants