facebook / mysql-5.6

Facebook's branch of the Oracle MySQL database. This includes MyRocks.
http://myrocks.io
Other
2.48k stars 714 forks source link

Support all of the system tables (except mysql.gtid_executed) in MyRocks #1428

Closed laurynas-biveinis closed 7 months ago

laurynas-biveinis commented 7 months ago
laurynas-biveinis commented 7 months ago

@luqun , this is a draft PR before I leave for the PTO. There are test failures which I haven't analyzed yet: rocksdb_dd_innodb.virtual_basic rocksdb_clone.remote_dml_gtid rocksdb.sys_tables_transactional_acl_tables

facebook-github-bot commented 7 months ago

@luqun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot commented 7 months ago

@laurynas-biveinis has updated the pull request. You must reimport the pull request before landing.

laurynas-biveinis commented 7 months ago

@luqun , ready for review

facebook-github-bot commented 7 months ago

@luqun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

luqun commented 7 months ago

also what's difference between rocksdb MTR test suites and rocksdb_dd_innodb MTR test suites? Previous I think rocksdb_dd_innodb MTR test suites is for innodb testcases runs under rocksdb DD, but now looks like some MTRs under rocksdb test suites also has similar pattern.

laurynas-biveinis commented 7 months ago

also what's difference between rocksdb MTR test suites and rocksdb_dd_innodb MTR test suites? Previous I think rocksdb_dd_innodb MTR test suites is for innodb testcases runs under rocksdb DD,

This is correct.

but now looks like some MTRs under rocksdb test suites also has similar pattern.

Which ones?

luqun commented 7 months ago

but now looks like some MTRs under rocksdb test suites also has similar pattern.

Which ones?

such as mysql-test/suite/rocksdb/t/sys_tables_mysql_upgrade_slave_master_info.test

laurynas-biveinis commented 7 months ago

but now looks like some MTRs under rocksdb test suites also has similar pattern.

Which ones?

such as mysql-test/suite/rocksdb/t/sys_tables_mysql_upgrade_slave_master_info.test

This is not an InnoDB-specific test, rather it was one of the general tests that regressed with a different DDSE

facebook-github-bot commented 7 months ago

This pull request has been merged in facebook/mysql-5.6@62658bc425c22bd21ce10914357eaad1d3966d8a.