winklemad commented on PR #1219: URL: https://github.com/apache/arrow-go/pull/1219#issuecomment-5491165504
Fixed in fe3b0ab. The final run end and the per-run normalization in the write loop now stay in the run-end type `T` and clamp against `T(inputData[i].Len())`, so nothing goes through `int` — the `MaxInt64` / logical-length-1 case no longer wraps to a negative value on 32-bit targets. Added `TestConcatRunEndEncodedInt64FinalRunEndClamp` for that scenario (it passes on a 64-bit host where `int` is 64-bit, and guards the 32-bit path). REE/concat suite + `go vet` are green. -- 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]
