Skip to content

Keep three integers after the comma#414

Open
gleu wants to merge 3 commits intomasterfrom
issue413
Open

Keep three integers after the comma#414
gleu wants to merge 3 commits intomasterfrom
issue413

Conversation

@gleu
Copy link
Copy Markdown
Collaborator

@gleu gleu commented Feb 19, 2026

Fixes #413.

@Krysztophe
Copy link
Copy Markdown
Collaborator

OK for me, except that I would remove another digit, 4 after the point seems weird

Before:

POSTGRES_STREAMING_DELTA CRITICAL: critical replay lag: 15.9287338256836MB for ...

After:

POSTGRES_STREAMING_DELTA CRITICAL: critical replay lag: 15.1022MB for ...

@gleu
Copy link
Copy Markdown
Collaborator Author

gleu commented Mar 26, 2026

Did some changes to only have two digits after the dot (looks better to me, and in line with what was asked on another issue. Can you check this? Thanks.

@gleu
Copy link
Copy Markdown
Collaborator Author

gleu commented Mar 26, 2026

I need to work on the tests :) but the fix should work anyway.

@gleu
Copy link
Copy Markdown
Collaborator Author

gleu commented Mar 27, 2026

Tests are working now. I kinda dislike the ".00" but I don't feel like working on it :)

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.

Streaming delta: useless accuracy

2 participants