Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Merge requests 144
    • Merge requests 144
  • 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
  • nexedi
  • erp5erp5
  • Merge requests
  • !2168

Open
Created Oct 02, 2025 by Jérome Perrin@jeromeOwner
  • Report abuse
Report abuse

Draft: Inventory API: Do not use regex for measure

  • Overview 6
  • 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
Reviewer
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