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

MDEV-28103 container /docker-entrypoint-initdb.d sql on master doesn't replicate #421

Conversation

grooverdan
Copy link
Member

Temporary testing image pushed to quay.io/danielgblack/mariadb-test:10.6-MDEV-28103

summary:

A user wants to initialize the master with a bunch of SQL to be
replicated doesn't get binary logged due to the --skip-log-bin
on the temporary server start.

The commit that this reverts added the skip-log-bin to speed up
timezones. A later commit 013d851
speed up the timezone initialization by disabling binary logs
when timezones where loaded.

The mariadb-upgrade does set SET SQL_LOG_BIN=0 by default.

As such:
Revert "MDEV-27074: disable log-bin during temp server start"

This reverts commit 146e09c.

…t replicate

A user wants to initialize the master with a bunch of SQL to be
replicated doesn't get binary logged due to the --skip-log-bin
on the temporary server start.

The commit that this reverts added the skip-log-bin to speed up
timezones. A later commit 013d851
speed up the timezone initialization by disabling binary logs
when timezones where loaded.

The mariadb-upgrade does set `SET SQL_LOG_BIN=0` by default.

As such:
Revert "MDEV-27074: disable log-bin during temp server start"

This reverts commit 146e09c.

Thanks Markus Mäkelä for the bug report.
@grooverdan grooverdan merged commit 5b93a88 into MariaDB:master Mar 17, 2022
@grooverdan grooverdan deleted the MDEV-28103-skip-binlog-docker_temp_server_start branch March 17, 2022 21:45
grooverdan added a commit to grooverdan/official-images that referenced this pull request Mar 17, 2022
A regression that a container master doesn't contain
the binary logs of its initdb.d data and cannot replicate this.

Fix details: MariaDB/mariadb-docker#421.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant