kenhuuu commented on PR #2738: URL: https://github.com/apache/tinkerpop/pull/2738#issuecomment-2307498975
> I run the above test on dotnet 6 without my change on MoveNext() and it produced `null`, so I tried to make it have the same behaviour on dotnet 8. Yea, generally, this makes sense to me since our policy is to prevent breaking changes. However, in this specific case, that old behavior was kind of undefined behavior and we've been OK with changing it in 3.6-dev recently (see https://github.com/apache/tinkerpop/pull/2424#issue-2064310991) I don't have a strong opinion on this topic because I'm not that familiar with C# so if no one else that is experienced in C# opposes it then I'm fine with supporting this PR as is. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
