Skip to content

fix: avoid seq scan when updating member identities#3817

Merged
skwowet merged 2 commits intomainfrom
hotfix-data-sink-worker
Feb 3, 2026
Merged

fix: avoid seq scan when updating member identities#3817
skwowet merged 2 commits intomainfrom
hotfix-data-sink-worker

Conversation

@skwowet
Copy link
Copy Markdown
Collaborator

@skwowet skwowet commented Feb 3, 2026

Note

Low Risk
Small SQL predicate change scoped to a single bulk update; main risk is unintentionally not updating rows that were expected to be restored from soft-delete.

Overview
MemberRepository.updateIdentities now filters its bulk UPDATE to only affect memberIdentities rows where deletedAt is null, preventing soft-deleted identities from being re-verified/modified during identity updates.

Written by Cursor Bugbot for commit aa61e3c. This will update automatically on new commits. Configure here.

@skwowet skwowet self-assigned this Feb 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 3, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conventional Commits FTW!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 3, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

2 similar comments
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 3, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 3, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@skwowet skwowet changed the title fix: updateIdentities to ignore soft-deleted member identities fix: avoid seq scan when updating member identities Feb 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 3, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@skwowet skwowet merged commit 78c8242 into main Feb 3, 2026
16 of 19 checks passed
@skwowet skwowet deleted the hotfix-data-sink-worker branch February 3, 2026 09:53
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