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.

Sign up for GitHub

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

Jump to bottom

Remove insert of backfill item #128

Merged
merged 1 commit into from
Feb 4, 2024
Merged

Remove insert of backfill item #128

merged 1 commit into from
Feb 4, 2024

Conversation

kespinola
Copy link
Collaborator

Issue

  • Ingestion slowed to a snail after release reintroduced the insertion of backfill items.
  • The select used was causing accessshare and excluselocks to be taken out on the table which prevents ingestion handlers from completing.

Change

  • Remove the insertion of the record since cl_audits_v2 is used now instead of it for tree backfilling.

@kespinola
fix: remove the insertion of backfill item it is no longer used for t…
28d390f 
…ree backfilling and causing excessive locks on db slowing ingestion processing
@kespinola kespinola merged commit 001655d into triton-build Feb 4, 2024
3 checks passed
@kespinola kespinola deleted the espi/remove-backfill-item-insert branch February 4, 2024 09:53
@kespinola kespinola mentioned this pull request Feb 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@kespinola