[
https://issues.apache.org/jira/browse/BEAM-6523?focusedWorklogId=191107&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-191107
]
ASF GitHub Bot logged work on BEAM-6523:
----------------------------------------
Author: ASF GitHub Bot
Created on: 28/Jan/19 16:28
Start Date: 28/Jan/19 16:28
Worklog Time Spent: 10m
Work Description: robertwb commented on pull request #7646: [BEAM-6523]
Add transcoding extension for Python flatten tests
URL: https://github.com/apache/beam/pull/7646#discussion_r251489518
##########
File path: sdks/python/apache_beam/runners/portability/fn_api_runner_test.py
##########
@@ -188,20 +188,25 @@ def test_pardo_windowed_side_inputs(self):
(9, list(range(7, 10)))]),
label='windowed')
- def test_flattened_side_input(self):
+ def test_flattened_side_input(self, with_transcoding=True):
with self.create_pipeline() as p:
main = p | 'main' >> beam.Create([None])
side1 = p | 'side1' >> beam.Create([('a', 1)])
side2 = p | 'side2' >> beam.Create([('b', 2)])
- side3 = p | 'side3' >> beam.Create([('b', 3)])
+ if with_transcoding:
+ # Also test non-matching coder types (transcoding required)
+ third_element = [('another_type')]
Review comment:
unneeded parens
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 191107)
Time Spent: 50m (was: 40m)
> Support transcoding for Flatten in Flink Runner
> -----------------------------------------------
>
> Key: BEAM-6523
> URL: https://issues.apache.org/jira/browse/BEAM-6523
> Project: Beam
> Issue Type: New Feature
> Components: runner-flink
> Reporter: Maximilian Michels
> Priority: Minor
> Time Spent: 50m
> Remaining Estimate: 0h
>
> The Beam documentation mentions that Flatten does not require coders of all
> inputs to be of the same type, as long as the result type is the same.
> However, the current implementation in the Flink Runner requires all coders
> to match.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
