Skip to content

Fix query to avoid nullable fields in select for update in processOldStreams job#1844

Merged
garrrikkotua merged 2 commits intomainfrom
fix/for-update-lock-in-webhooks
Nov 15, 2023
Merged

Fix query to avoid nullable fields in select for update in processOldStreams job#1844
garrrikkotua merged 2 commits intomainfrom
fix/for-update-lock-in-webhooks

Conversation

@garrrikkotua
Copy link
Copy Markdown
Contributor

@garrrikkotua garrrikkotua commented Nov 15, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 0390bf5

Improved the performance of the integration stream worker by optimizing the SQL query in integrationStream.repo.ts.

🤖 Generated by Copilot at 0390bf5

We're sailing on the integrationStream tonight
We've got to fetch and lock the webhooks right
So pull the rope and sing along with me
We've simplified the SQL query

Why

How

🤖 Generated by Copilot at 0390bf5

  • Simplify and format SQL query to fetch and lock pending incoming webhooks (link, link) in integrationStream.repo.ts

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@garrrikkotua garrrikkotua merged commit b931f0a into main Nov 15, 2023
@garrrikkotua garrrikkotua deleted the fix/for-update-lock-in-webhooks branch November 15, 2023 11:06
sausage-todd pushed a commit that referenced this pull request Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant