Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • W wendelin.core
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 5
    • Merge requests 5
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • nexedinexedi
  • wendelin.core
  • Merge requests
  • !29

bigfile/zodb: Apply auto format as default only in WCFS mode

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Kirill Smelkov requested to merge kirr/wendelin.core:y/auto-only-wcfs into master Jul 22, 2024
  • Overview 5
  • Commits 1
  • Pipelines 0
  • Changes 1

This semantically reverts 99f262dd (bigfile/zodb: Make auto format the default) for wendelin.core-1 mode because in non-WCFS mode there are known problems with data corruption on BTree topology changes(*) and auto mode actually does change those topologies with first setting ZBigFile[blk] -> ZBlk1 and then updating the same block to point to ZBlk0 object.

Avoid pressuring those problems and use auto as default only in WCFS mode that should handle invalidations with all those BTree topology changes well.

The patch is based on suggestion by Levin Zimmermann: https://lab.nexedi.com/nexedi/wendelin.core/-/merge_requests/20#note_212405

We have to move _default_use_wcfs because now it is invoked at module import time and needs to be already defined at the time of the call.

(*) see https://lab.nexedi.com/nexedi/wendelin.core/commit/8c32c9f6 for details.

/cc @levin.zimmermann, @klaus, @Tyagov

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: y/auto-only-wcfs
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7