Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved performance for filter strings with multiple neq statements #20198

Merged
merged 1 commit into from
May 13, 2024

Conversation

9larsons
Copy link
Contributor

ref https://linear.app/tryghost/issue/CFR-27

  • updated packages to include performance improvement for NQL filter strings including multiple neq filters for the same resource
  • bumped bookshelf-plugins
  • bumped NQL versions

We identified a performance fix that allows us to combine not equal (neq) filters for the same resource in a logically-equivalent way that also has far more performant resulting SQL.

We're effectively automatically combining strings like 'tag:-tag1+tag:-tag2` into 'tag:-[tag1,tag2]'.

ref https://linear.app/tryghost/issue/CFR-27
- updated packages to include performance improvement for NQL filter strings including multiple neq filters for the same resource
- bumped `bookshelf-plugins`
- bumped NQL versions

We identified a performance fix that allows us to combine not equal (neq) filters for the same resource in a logically-equivalent way that also has far more performant resulting SQL.

We're effectively automatically combining strings like 'tag:-tag1+tag:-tag2` into 'tag:-[tag1,tag2]'.
@github-actions github-actions bot added the affects:admin Anything relating to Ghost Admin label May 13, 2024
@9larsons 9larsons merged commit 842290c into main May 13, 2024
22 checks passed
@9larsons 9larsons deleted the nql-get-performance-fix branch May 13, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant