rainwoodman commented on a change in pull request #16351:
URL: https://github.com/apache/beam/pull/16351#discussion_r815364186
##########
File path: sdks/python/apache_beam/transforms/core.py
##########
@@ -1714,14 +1713,20 @@ def MapTuple(fn, *args, **kwargs): # pylint:
disable=invalid-name
# Proxy the type-hint information from the original function to this new
# wrapped function.
- type_hints = get_type_hints(fn)
+ type_hints = get_type_hints(fn).with_defaults(
+ typehints.decorators.IOTypeHints.from_callable(fn))
if type_hints.input_types is not None:
- wrapper = with_input_types(
- *type_hints.input_types[0], **type_hints.input_types[1])(
- wrapper)
+ # FIXME(feyu): ignore input hints, as we do not have enough
Review comment:
Not yet. I'll file an issue and update the fixme tag. A fix is likely
non-trivial, because it seems like our data model is insufficient (as the
comment noted)
--
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]
