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

Missing indexes, script improvements, fixes #1500

Merged
merged 4 commits into from
Sep 15, 2023

Conversation

themarolt
Copy link
Contributor

@themarolt themarolt commented Sep 15, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 4aa70ff

This pull request improves the data synchronization and processing of organizations and segment activity channels. It changes the field used for sorting organizations in the search_sync_worker service, creates a new index for the segmentActivityChannels table in the backend database, and adds a new script to the data_sink_worker service to process the data sink results.

🤖 Generated by Copilot at 4aa70ff

Sing, O Muse, of the skillful coder who changed the index
Of segmentActivityChannels, the table of many platforms
And who made a new script, script:process-results, to run
The tasks of the data_sink_worker, the service of data streams

Why

How

🤖 Generated by Copilot at 4aa70ff

  • Create a new index on segmentActivityChannels table to improve query performance (link, link)
  • Add a new script to process data sink results and update segment activity channels (link)
  • Replace date_joinedAt with date_createdAt in OrganizationSyncService class to use more accurate and consistent data for sorting, filtering, and pagination of organizations to be synced to OpenSearch index (link, link, link, 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 Bug Created by Linear-GitHub Sync Improvement Created by Linear-GitHub Sync labels Sep 15, 2023
@themarolt themarolt merged commit f09901f into main Sep 15, 2023
7 checks passed
@themarolt themarolt deleted the hotfix/missing-indexes-and-script-fixes branch September 15, 2023 07:11
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 Improvement Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants