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

services/horizon/ingest: fix 'Reingestion done' log message to not include prepare range duration #5156

Closed
sreuland opened this issue Jan 8, 2024 · 1 comment
Closed

services/horizon/ingest: fix 'Reingestion done' log message to not include prepare range duration #5156

sreuland opened this issue Jan 8, 2024 · 1 comment
Assignees
@sreuland
Labels
bug horizon ingest New ingestion system performance issues aimed at improving performance

Comments

@sreuland
Copy link
Contributor

sreuland commented Jan 8, 2024

What version are you using?

release-horizon-v2.28.0 branch

What did you do?

ran forward ingestion

What did you expect to see?

'Reingestion done' log message duration value should equate to same timeframe as prior 2.27.0

What did you see instead?

'Reingestion done' log message duration value included extra time for prepare range duration which makes it incompatible to values from prior releases.
@sreuland sreuland added horizon bug performance issues aimed at improving performance labels Jan 8, 2024
@sreuland sreuland moved this from Backlog to Current Sprint in Platform Scrum Jan 8, 2024
@sreuland sreuland added the ingest New ingestion system label Jan 8, 2024
@sreuland sreuland self-assigned this Jan 8, 2024
@sreuland sreuland mentioned this issue Jan 10, 2024
Closed
@sreuland sreuland moved this from Current Sprint to In Progress in Platform Scrum Jan 10, 2024
sreuland added a commit to sreuland/go that referenced this issue Jan 11, 2024
@sreuland
stellar#5156: do not include range prep time in 'Reingestion done' lo…
c3eed80 
…gged duration
@sreuland sreuland mentioned this issue Jan 11, 2024
Merged
7 tasks
@sreuland sreuland moved this from In Progress to Needs Review in Platform Scrum Jan 11, 2024
sreuland added a commit that referenced this issue Jan 12, 2024
@sreuland
#5156: do not include range prep time in 'Reingestion done' logged du…
15324b7 
…ration (#5159)
@sreuland
Copy link
Contributor Author

merged #5159

@github-project-automation github-project-automation bot moved this from Needs Review to Done in Platform Scrum Jan 12, 2024
sreuland added a commit to sreuland/go that referenced this issue Jan 29, 2024
@sreuland
stellar#5156: do not include range prep time in 'Reingestion done' lo…
401f692 
…gged duration (stellar#5159)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sreuland sreuland

Labels
bug horizon ingest New ingestion system performance issues aimed at improving performance
Projects
Platform Scrum
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sreuland