lidavidm commented on pull request #10412: URL: https://github.com/apache/arrow/pull/10412#issuecomment-852980554
It's in a struct because it's a different length, yes. We could make it a vector kernel instead of a scalar kernel and then the replacements could be passed as another argument. I'm not sure which would be more useful though. For the name, replace_with_mask etc. sound good. CC @nirandaperera so he's aware for ARROW-9431 as well. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
