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

PS-9218 merge: Merge MySQL 8.4.0 (MTR test fixes) #5355

Conversation

percona-ysorokin
Copy link
Collaborator

oleksandr-kachan and others added 2 commits July 16, 2024 23:26
https://perconadev.atlassian.net/browse/PS-9218

- removed enable/disable_reconnect
- fixed path to included files ('extra/' -> 'common/')
- removed 'rocksdb.upgrade_parts_from_prev_ver' test, it checked upgrades
  from previous (non-native) partitioning format in 5.7
- updated rocksdb_rpl.rpl_rocksdb_stm_mixed_auto_increment and
  rocksdb_rpl.rpl_rocksdb_row_auto_increment tests to use
  rpl_auto_increment_innodb.test internally as rpl_auto_increment.test
  is now deleted
- updated and renamed rocksdb_rpl.rpl_rocksdb_extra_col_slave and
  rocksdb_rpl.rpl_rocksdb_extra_col_master tests
- deprecated LOW_PRIORITY WRITE changed to WRITE in rocksdb.lock
- removed ENGINE=xxx from DROP TABLESPACE statement in rocksdb.create_table
- re-record innodb.log_encrypt_kill test
- updated percona.utility_user test to mask thread ID in ER_KILL_DENIED_ERROR
  error string
- in rocksdb.autoinc_vars removed checks for DOUBLE AUTO_INCREMENT
  as support for it was removed in this release
- in rocksdb_rpl.rpl_rocksdb_row_img_idx_full, rocksdb_rpl.rpl_rocksdb_row_img_idx_min
  and rocksdb_rpl.rpl_rocksdb_row_img_idx_noblob
  binlog_transaction_dependency_tracking=COMMIT_ORDER changed to
  binlog_transaction_dependency_history_size=1 as COMMIT_ORDER was removed by
  upstream.
…connection_configuration

[Post fix]

Fixing test rpl_log_info_repository_persistence_require_row that was
not accounting for the group replication recovery channel that is now
shown when configured.

Change-Id: I514e8f03536d928f884505cca7be130fe8285b4d
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (1/1)

@@ -6,7 +6,7 @@
--connection master
--let $master_host = 127.0.0.1

CREATE USER 'rocksdb_stress_test'@'localhost' IDENTIFIED WITH 'mysql_native_password';
CREATE USER 'rocksdb_stress_test'@'localhost' IDENTIFIED WITH 'caching_sha2_password';
GRANT ALL ON *.* TO 'rocksdb_stress_test'@'localhost';

Choose a reason for hiding this comment

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

⚠️ clang-diagnostic-error ⚠️
unknown type name GRANT

@@ -6,7 +6,7 @@
--connection master
--let $master_host = 127.0.0.1

CREATE USER 'rocksdb_stress_test'@'localhost' IDENTIFIED WITH 'mysql_native_password';
CREATE USER 'rocksdb_stress_test'@'localhost' IDENTIFIED WITH 'caching_sha2_password';
GRANT ALL ON *.* TO 'rocksdb_stress_test'@'localhost';

Choose a reason for hiding this comment

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

⚠️ clang-diagnostic-error ⚠️
expected ; after top level declarator

Suggested change
GRANT ALL ON *.* TO 'rocksdb_stress_test'@'localhost';
GRANT ALL; ON *.* TO 'rocksdb_stress_test'@'localhost';

@@ -6,7 +6,7 @@
--connection master
--let $master_host = 127.0.0.1

CREATE USER 'rocksdb_stress_test'@'localhost' IDENTIFIED WITH 'mysql_native_password';
CREATE USER 'rocksdb_stress_test'@'localhost' IDENTIFIED WITH 'caching_sha2_password';
GRANT ALL ON *.* TO 'rocksdb_stress_test'@'localhost';

let $exec =

Choose a reason for hiding this comment

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

⚠️ clang-diagnostic-error ⚠️
unknown type name let

@@ -21,7 +21,6 @@

--exec rm $MYSQL_TMP_DIR/load_generator.log

Choose a reason for hiding this comment

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

⚠️ clang-diagnostic-error ⚠️
expected unqualified-id

@@ -21,7 +21,6 @@

--exec rm $MYSQL_TMP_DIR/load_generator.log

enable_reconnect;
source include/wait_until_connected_again.inc;

--let $master_checksum = query_get_value(CHECKSUM TABLE $table, Checksum, 1)

Choose a reason for hiding this comment

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

⚠️ clang-diagnostic-error ⚠️
expected unqualified-id

@percona-ysorokin percona-ysorokin merged commit a81167e into percona:release-8.4.0-1 Jul 17, 2024
9 of 11 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.

3 participants