AnkitAhlawat7742 opened a new pull request, #49950: URL: https://github.com/apache/arrow/pull/49950
Fix #47447 ### Rationale for this change replace_with_mask crashes when called with null type arrays because a code path incorrectly returns Status::OK() from a function with return type Result<int64_t>. This PR fixes the issue by ensuring the function returns a valid int64_t value instead of Status::OK() for successful executon ### What changes are included in this PR? 1. Fixed the replace_with_mask kernel implementation for null type arrays. 2. Replaced the invalid Status::OK() return path with a valid int64_t result. 3. Prevented construction of Result<int64_t> with an OK Status. ### Are these changes tested? Yes, manually run the test case ### Are there any user-facing changes? No **This PR contains a "Critical Fix".** This change fixes a crash in replace_with_mask when operating on null type arrays. The crash occurs due to an invalid successful return path internally constructing Result<T> with Status::OK(). -- 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]
