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
  • !2168

Draft: Inventory API: Do not use regex for measure

  • Review changes

  • Download
  • Patches
  • Plain diff
Open Jérome Perrin requested to merge fix/inventory_api_no_regexp into master Oct 02, 2025
  • Overview 7
  • Commits 1
  • Pipelines 1
  • Changes 7

concat(stock.variation_text,'\n') REGEXP measure.variation is slow. This introduces an alternative approach of indexing the first 4 variations directly in columns and use a traditional join instead or a regexp match. In a real production case of a 2 seconds query, this was one order of magnitude faster.

This has some limitations:

  • it does not support more than 4 variations
  • it does not support optional variations

The previous regexp based implementation is kept in this patch, so that it's easy to rollback this change if side effects are found.

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