Skip to content

Conversation

themarolt
Copy link
Contributor

@themarolt themarolt commented Sep 13, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 84bea03

This pull request refactors the logic for fetching and processing integration streams in the IntegrationStreamWorker service. It uses cursor-based pagination to improve performance and avoid inconsistencies. It also simplifies the stream state management by removing the PROCESSING value from the IntegrationStreamState enum.

copilot:poem

Why

How

🤖 Generated by Copilot at 84bea03

  • Replace offset-based pagination with cursor-based pagination for fetching pending streams from the database (link, link, link)
  • Remove unused PROCESSING value from IntegrationStreamState enum in integrations.ts (link)
  • Remove unused markStreamInProgress method of IntegrationStreamRepo class in integrationStream.repo.ts (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots 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.

@themarolt themarolt added the Bug Created by Linear-GitHub Sync label Sep 13, 2023
@themarolt themarolt merged commit dcaeec2 into main Sep 18, 2023
@themarolt themarolt deleted the enhancement/remove-processing-stream-state branch September 18, 2023 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant