zeroshade commented on issue #635: URL: https://github.com/apache/arrow-go/issues/635#issuecomment-5501437303
Closing this as stale — the version this was filed against is several majors behind now. arrow-go is on `substrait-io/substrait-go/v8 v8.1.1` on main, versus the v7.2.2 discussed here. Unsigned Arrow types are given Substrait user-defined extension types (`arrow/compute/exprs/types.go`), so `arrow.UINT8` comparisons should work with an explicit CAST, which you confirmed covers most of your cases. The one open thread was the `uint64` literal above `math.MaxInt64` edge case, where I asked whether the user-defined types covered it (https://github.com/apache/arrow-go/issues/635#issuecomment-2813237723-style follow-up on 2026-05-05) and we didn't hear back. @Sunyue if that edge case is still a problem on v8, please open a fresh issue with a small reproducer against the current release — a new issue will get better attention than this one, since the v7 context no longer applies. Thanks for the original report. -- 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]
