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

DM-47512: Upgrade MariaDB in Qserv to the latest stable version 11.4.4 #878

Merged
merged 6 commits into from
Nov 20, 2024

Conversation

iagaponenko
Copy link
Contributor

No description provided.

@iagaponenko iagaponenko force-pushed the tickets/DM-47512 branch 2 times, most recently from 4883d39 to 1a07b25 Compare November 16, 2024 05:03
@iagaponenko iagaponenko changed the title DM-47512: Upgrade MariaDB in Qserv to the latest stable version 11.4.3 DM-47512: Upgrade MariaDB in Qserv to the latest stable version 11.4.4 Nov 19, 2024
The change was made by running the latest version of clang-format
in the build container. Apparently, the tool pulled into the latest
version of the container has different rules for the C++ code.
The older verson of the tool was:
"clang-format version 17.0.6 (AlmaLinux OS Foundation 17.0.6-5.el9)"
The new version is now:
"clang-format version 18.1.8 (AlmaLinux OS Foundation 18.1.8-3.el9)"
Also upgraded to the newer release 0.8.5.lsst8 of mysqlproxy which supports
MariaDB 11.
Eliminated MySQL meta-commands that were present in the schema files.
These commands were produced by "mysqldump" many years ago
when the original schemas were sampled from some older MySQL version 5.*.
It turns out, some of the metacommands are no longer supported by MariaDB 11.4.*.
Besides, these meta-comands are useless in the context of the integration tests.
…egration tests

The change was introduced after migration to MariaDB 11.4.4 which
has a few seconds delay before it starts listening for connections.
If the integration test starts reading from the database right away then
it fails.
@fritzm fritzm self-requested a review November 20, 2024 16:07
Copy link
Contributor

@fritzm fritzm left a comment

Choose a reason for hiding this comment

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

LGTM -- thanks!

@iagaponenko iagaponenko merged commit 82715e6 into main Nov 20, 2024
15 checks passed
@iagaponenko iagaponenko deleted the tickets/DM-47512 branch November 20, 2024 18:09
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.

2 participants