[ 
https://issues.apache.org/jira/browse/BEAM-11196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17230185#comment-17230185
 ] 

Yifan Mai commented on BEAM-11196:
----------------------------------

[https://github.com/apache/beam/pull/13202] introduces a bug where the parent 
computation can cause loops, because the lowest common ancestor algorithm 
considers a transform to be its own parent. Fusing a single transform A will 
result cause the fused stage's parent to be set to A. Fusing a transform A with 
a descendant B will also cause fused stage's parent to be set to A. To fix 
this, if the parent of the fused stage would be one of transforms to be fused, 
set the parent to the parent of that transform instead.

> Composite transform structure not preserved with fusion.
> --------------------------------------------------------
>
>                 Key: BEAM-11196
>                 URL: https://issues.apache.org/jira/browse/BEAM-11196
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-py-core
>            Reporter: Robert Bradshaw
>            Assignee: Yifan Mai
>            Priority: P2
>             Fix For: 2.26.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> This causes problems with combiner packing, among other things.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to