Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1844

WIP: Introducing READ-COMMITTED isolation connector

  • Review changes

  • Download
  • Patches
  • Plain diff
Closed Kazuhiko Shiozaki requested to merge feat/mariadb-10.11 into master Nov 15, 2023
  • Overview 8
  • Commits 30
  • Pipelines 0
  • Changes 150

This branch is for MariaDB >= 10.5 that does no longer have innodb_locks_unsafe_for_binlog.

To achieve a good performance in parallel processing, this branch uses following connections.

  • cmf_activity_sql_connection: READ-COMMITTED isolation
  • erp5_sql_connection: REPEATABLE-READ isolation
  • erp5_sql_read_committed_connection: [NEW] READ-COMMITTED isolation
  • erp5_sql_transactionless_connection: not using transaction

Basically, new erp5_sql_read_committed_connection is used for INSERT/DELETE/REPLACE operation. For certain cases for which we are sure that READ-COMMITTED isolation is fine, we can use it for SELECT operation as well, for example alarm table.

Also for certain cases for which we don't join with other tables, using the default erp5_sql_connection connection for everything is easier, for example syncml table.

Note that both READ-COMMITTED transaction and REPEATABLE-READ transaction start right after the Zope transaction starts, and READ-COMMITTED transaction's result cannot be fetched by REPEATABLE-READ transaction. Some unit tests failed by this reason and simply calling self.commit() will re-starts both MariaDB transactions.

Edited Oct 04, 2024 by Kazuhiko Shiozaki
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feat/mariadb-10.11
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7