You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.
After the linked general database abstraction issues have been completed it should be possible to create a new PDO adapter and schema definition to trial running Matomo entirely on TiDB. an open source distributed MySQL compatible database.
This change will allow for long term suitability testing of TiDB as an alternative high-performance database backend for Matomo.
Create a new PDO adapter core/Db/Adapter/Pdo/Tidb based on core/Db/Adapter/Pdo/Mysql.php.
Fully implement all requirements of core/Db/AdapterInterface.
Implement getLock() using the approach in the lockalt branch.
supportsWindowFunctions() to return true.
preferSubqueries() to return true.
Create a new OmniFixture dump compatible with the TiDB schema and return it's filename with `getOmniFixtureFilename()'.
Check and set the global sql_mode to disable strict grouping operators.
Create a new schema class at core/Db/Schema/Tidb
Use AUTO_RANDOM Primary keys.
Set TiFlash replicas for all log_xxx tables, this may need config settings for clusters with multiple replicas.
Create TiDB specific indexes.
TEXT columns cannot have default values, remove the defaults.
Related general database abstraction issues:
#19643: Abstract the database get_lock function to PDO adapter #19646: Support option for ranking queries to use window functions #19647: Abstract database specific tests and fixture code #19648: Abstract database specific helper methods to PDO adapter #19649: Support database specific DDL in migrations #19650: Use sequences instead of auto increment keys for user facing identifiers #19651: Support optional sub-query substitution for COUNT(DISTINCT x) queries
The text was updated successfully, but these errors were encountered:
@sgiehl This would be covered by issue #19648. My suggested approach there is almost identical to your solution for #18371, the only difference is you've implemented the abstraction using a db schema class whereas I was thinking to have statement code in a db specific PDO adapter and just have schema changes in the schema class.
PDO adapter descendants get access to override some lower level functions such as query() which might be needed if we implement a multi-tenancy solution at some point and need to enforce query safe guards. Also some of the existing MySQL PDO adapter code is already too MySQL specific to work with TiDB, SingleStore, etc, which don't have InnoDB tables at all, different default ports and different version numbering.
We could allow database choice during installation and then set the existing config.ini.php adapter and schema settings to determine the schema and PDO adapter classes to use.
It looks like #18371 is already compatible with this approach. Maybe it could be finished at the same time #19648 is implemented? It'd be a good test of the abstraction structure.
Therefore, #19650may no longer be required for TiDB support. Note however that tables with AUTO_INCREMENT primary keys are still prone to write ( and potentially read ) hotspots. Please feel free to reach out if you want to discuss further.