[
https://issues.apache.org/jira/browse/TINKERPOP-3161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18020778#comment-18020778
]
ASF GitHub Bot commented on TINKERPOP-3161:
-------------------------------------------
andreachild commented on PR #3206:
URL: https://github.com/apache/tinkerpop/pull/3206#issuecomment-3300498738
> I think this probably warrants a quick shout out in the upgrade docs
warning .Net users to either switch to `sbyte` or shorts. Other than that, my
vote is VOTE +1
I agree, needs to be added to upgrade docs.
VOTE +1 pending upgrade docs
> Switched to signed bytes in .net
> --------------------------------
>
> Key: TINKERPOP-3161
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3161
> Project: TinkerPop
> Issue Type: Improvement
> Components: dotnet
> Affects Versions: 3.7.3
> Reporter: Cole Greer
> Priority: Critical
> Labels: breaking
>
> The `Byte` type in our serializers represents a signed 8-bit integer (as this
> is what byte in Java represents), however this currently maps to `byte` in
> .net, which is an unsigned value. The .net serializers should be updated to
> use `sbyte` instead, to be consistent with the rest of TinkerPop.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
