Skip to content

link-index-updater: Optimize and add more logging #1215

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
Merged
merged 1 commit into from
May 5, 2025
Merged

link-index-updater: Optimize and add more logging #1215

merged 1 commit into from
May 5, 2025

Conversation

reakaleek
Copy link
Member

@reakaleek reakaleek commented May 5, 2025 •
edited
Loading

Changes

  • Optimization: Don't execute an S3 put operation when the link index file did not change.
  • Trying to understand why messages are not sent back to the queue and reprocessed by the lambda. I'm 99% sure this was working in my initial testing.

@reakaleek
link-index-updater: More logging
91d1fb0 
Trying to understand why messages are not sent back to the queue and reprocessed by the lambda
@reakaleek reakaleek requested a review from a team as a code owner May 5, 2025 12:47
@reakaleek reakaleek changed the title link-index-updater: More logging link-index-updater: Optimize and add more logging May 5, 2025
@reakaleek reakaleek self-assigned this May 5, 2025
@reakaleek reakaleek merged commit 05dfe82 into main May 5, 2025
14 of 15 checks passed
@reakaleek reakaleek deleted the feature/link-index-updater-logging branch May 5, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mpdreamz Mpdreamz Mpdreamz approved these changes

Assignees

@reakaleek reakaleek

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@reakaleek @Mpdreamz