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

Only log once at start of table scan for TDL-21257 #48

Merged
merged 2 commits into from
Dec 12, 2022
Merged

Only log once at start of table scan for TDL-21257 #48

merged 2 commits into from
Dec 12, 2022

Conversation

dsprayberry
Copy link
Contributor

Description of change

Decreases the frequency with which we write logs while scanning table in Dynamo's full-table method.

Manual QA steps

  • Tested in prod with PR-Alpha

Risks

  • Low

Rollback steps

  • revert this branch

@dsprayberry dsprayberry changed the title only log once at start ot table scan Only log once at start of table scan for TDL-21257 Dec 12, 2022
@dsprayberry dsprayberry merged commit 9059540 into master Dec 12, 2022
@dsprayberry dsprayberry deleted the TDL-21257/reduce-logging branch December 12, 2022 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leslievandemark leslievandemark leslievandemark approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dsprayberry @leslievandemark