[ https://issues.apache.org/jira/browse/BEAM-8224?focusedWorklogId=313296&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-313296 ]
ASF GitHub Bot logged work on BEAM-8224: ---------------------------------------- Author: ASF GitHub Bot Created on: 16/Sep/19 20:50 Start Date: 16/Sep/19 20:50 Worklog Time Spent: 10m Work Description: udim commented on issue #9563: [BEAM-8224] Fix bug in _fn_takes_side_inputs URL: https://github.com/apache/beam/pull/9563#issuecomment-531952821 trivial_inference.py defaults to Any in 3.6, 3.7 since it doesn't support kwargs (added by the wrapper). In 2.7, 3.5 these are supported, but since side-input hints are not passed along the inferred output type is str, and the coder for that fails to encode a tuple. Even if we did pass side-input hints, there is still missing logic that would bind the arguments for the var_positional into a single tuple (like getcallargs or signature.bind). Perhaps we should skip trivial_inference when side-inputs are present. ---------------------------------------------------------------- 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 Issue Time Tracking ------------------- Worklog Id: (was: 313296) Time Spent: 2h 40m (was: 2.5h) > The way we inspect arguments in Python does not work for lambda *_: ... > ----------------------------------------------------------------------- > > Key: BEAM-8224 > URL: https://issues.apache.org/jira/browse/BEAM-8224 > Project: Beam > Issue Type: Bug > Components: sdk-py-core > Reporter: Yueyang Qiu > Assignee: Udi Meiri > Priority: Blocker > Fix For: 2.16.0 > > Time Spent: 2h 40m > Remaining Estimate: 0h > > We need to make it work and create a test for this case. > > Assigning to Udi who authored > [https://github.com/apache/beam/pull/9283|https://www.google.com/url?q=https://github.com/apache/beam/pull/9283&sa=D&usg=AFQjCNETtLzhe9jPfGzLHJb2N4Sle-FuNw]. -- This message was sent by Atlassian Jira (v8.3.2#803003)