rpuch commented on PR #4777: URL: https://github.com/apache/ignite-3/pull/4777#issuecomment-2493892093
> Does it make sense to write updateToken as VarInt ? Can we be sure it never be a kind of timestamp or large value? I'm not sure, I was looking at the current situation. Are there any known plans of switching tokens from revisions to something (probably, less suitable for varints)? I'm not sure whether we should stick to varints here, to be honest, as they are used scarcely. If you believe just `long` is safer, let me know, I will change it back. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
