rpuch commented on PR #4777: URL: https://github.com/apache/ignite-3/pull/4777#issuecomment-2493884524
> Why don't just let DataOutput implementation take care of compressing numerics in writeInt/writeLong method? Not for every int/long does it make sense to write them as a varint. For example, our varints are only space-effective for n >= -1, and for most negative values they are worse than regular ints. Also, if a field is not guaranteed to contain a small value most of the time (which is true for collection lengths and ordinals, like table IDs), regular ints/longs might be better. So it seems that the programmer should decide whether to use a regular int or a varint. -- 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
