AntoinePrv commented on code in PR #46963:
URL: https://github.com/apache/arrow/pull/46963#discussion_r2179495090
##########
cpp/src/arrow/util/dispatch_internal.h:
##########
@@ -76,7 +76,8 @@ class DynamicDispatch {
protected:
// Use the Implementation with the highest DispatchLevel
- void Resolve(const std::vector<Implementation>& implementations) {
+ template <typename Range>
+ void Resolve(const Range& implementations) {
Review Comment:
I don't think it changes anything since we are never moving the object
inside the implementation.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
