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.

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

Update jsm.go #1746

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update jsm.go #1746

wants to merge 1 commit into from

Conversation

SinTod
Copy link

@SinTod SinTod commented Dec 4, 2024

fix arm32 device NumDeleted value overflow

fix arm32 device NumDeleted value overflow
@piotrpio
Copy link
Collaborator

piotrpio commented Dec 4, 2024

Hello @SinTod, thank you for the contribution. You're correct about this being a bug, but I'm not sure we can merge this PR as this would be a breaking change. Also, this is the same struct as in nats-server, so if we were to introduce this breaking change at some point, we would need to do this in the server as well (we cannot have int on the server and uint in the client, what if the server starts sending e.g. -1 at some point - for whatever reason).

@SinTod
Copy link
Author

SinTod commented Dec 5, 2024

hi @piotrpio If i change NumDeleted to int64?

@SinTod
Copy link
Author

SinTod commented Dec 5, 2024

@piotrpio btw, can I erase this value from nats server ?

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.

2 participants