alamb commented on PR #3839: URL: https://github.com/apache/arrow-rs/pull/3839#issuecomment-1464923448
> I would like to understand the use-case more before merging this, I wonder if it is something like flushing the cache, or being able to take some different action if the item is not in the cache 🤔 > I'm not a massive fan of breaking encapsulation in this way, it undermines the decorator pattern, and would rather avoid making a breaking change if possible I agree finding some way to avoid `as_any` might be preferable in most circumstances, but I don't see that as a reason to not add it to `as_any` if that is how the downstream project wants to structure their code. In other words I see no harm in adding this API as it allows significant flexibility even if it is not a preferred pattern for some projects -- 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]
