Skip to content

PBS-37 feature: Combine binlog_server with minimysql_server into a single executable (part 1)#157

Merged
percona-ysorokin merged 1 commit into
Percona-Lab:mainfrom
percona-ysorokin:combined_binary
Jul 1, 2026
Merged

PBS-37 feature: Combine binlog_server with minimysql_server into a single executable (part 1)#157
percona-ysorokin merged 1 commit into
Percona-Lab:mainfrom
percona-ysorokin:combined_binary

Conversation

@percona-ysorokin

Copy link
Copy Markdown
Collaborator

https://perconadev.atlassian.net/browse/PBS-37

Refactored 'minimysql_server' code:

  • Extracted frame-based network operations into 'minimysql/network_io_operations.hpp'
  • Extracted main MySQL protocol logic into 'minimysql/network_service.hpp'
  • Extracted sample binlog message data into 'minimysql/sample_event_collection.hpp'

Where possible we now use 'boost/asio/ts/netfwd.hpp' for boost::asio forward declarations.

Reworked the way how we suppress known false-positive warnings - we now always use '#pragma GCC diagnostic ignored' for individual headers instead of setting a '-Wno-error=...' flag on a whole compilation unit.

…ngle executable (part 1)

https://perconadev.atlassian.net/browse/PBS-37

Refactored 'minimysql_server' code:
* Extracted frame-based network operations into
  'minimysql/network_io_operations.hpp'
* Extracted main MySQL protocol logic into
  'minimysql/network_service.hpp'
- Extracted sample binlog message data into
  'minimysql/sample_event_collection.hpp'

Where possible we now use 'boost/asio/ts/netfwd.hpp' for boost::asio forward
declarations.

Reworked the way how we suppress known false-positive warnings - we now
always use '#pragma GCC diagnostic ignored' for individual headers instead of
setting a '-Wno-error=...' flag on a whole compilation unit.
@percona-ysorokin percona-ysorokin merged commit 9362b55 into Percona-Lab:main Jul 1, 2026
8 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.

1 participant