LuciferYang opened a new pull request, #13548: URL: https://github.com/apache/gluten/pull/13548
## What changes are proposed in this pull request? Removes two pieces of native code that are never used: - The two-argument `arrowGetOrThrow(arrow::Result<T>, const std::string&)` in `cpp/core/utils/ArrowStatus.h`. It has no callers, and it calls `ThrowPendingException`, which is not declared anywhere. - `VeloxHashShuffleWriter::complexTypeFlushBuffer_`. It was only ever resized to null pointers, so both complex-type flush paths always allocated a new buffer and the reuse branch never ran. The flush paths now allocate the buffer directly, which is what they already did, and the loop that added the member's capacity to `estimatePartitionBufferBytes` is gone, since it always added nothing. No behavior change. Fixes #13542. ## How was this patch tested? No new tests, since nothing observable changes. ## Was this patch authored or co-authored using generative AI tooling? Yes. Generated-by: Claude Opus. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
