Reranko05 commented on PR #49951: URL: https://github.com/apache/arrow/pull/49951#issuecomment-4627535661
@kou I'd appreciate your review of this PR when you get a chance. Root cause: the decimal-specific `VisitInExpression()` path embeds the `InHolder` address directly into generated object code. When Gandiva reuses cached object code, that pointer can become stale and cause a crash. Fix: load the holder dynamically from `arg_holder_ptrs_` at runtime, matching the existing generic `InExpression` implementation. Test: `TestInDecimalCached` forces a cache hit on the second `Filter::Make()` call and verifies correct evaluation from cached object code. This issue was originally reported in #39784 and a prior fix attempt (#39785) was never merged. The remaining failing CI job appears unrelated to the Gandiva changes. Thanks! -- 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]
