[GitHub] [arrow] js8544 commented on issue #35863: [C++] Can InputType support nested types in the future?

["js8544 (via GitHub)" <gi...@apache.org>]

js8544 commented on issue #35863:
URL: https://github.com/apache/arrow/issues/35863#issuecomment-1586193221

   > > AvgFinalMatcher
   > 
   > so, how to pass type_id?
   > 
   > original code,
   > 
   > ```
   > InputType(std::shared_ptr<TypeMatcher> type_matcher) {}
   > ```
   > 
   > now code：
   > 
   > ```
   > InputType(std::shared_ptr<TypeMatcher> type_matcher,  Type::type type_id_) {}
   > ```
   > 
   > This kind of interface is really needed.
   
   I think weston's point is that you can pass the type_id to your custom matcher class that inherits `TypeMatcher`, not necessarily to `InputType`.
   
   


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org