[
https://issues.apache.org/jira/browse/BEAM-6327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17105627#comment-17105627
]
Luke Cwik commented on BEAM-6327:
---------------------------------
I think this shouldn't be considered "trimming" but native transform
replacement and have crafted https://github.com/apache/beam/pull/11670 which
repurposes this as such. The version that was in the trimmer was a trivial
implementation that covers many scenarios where the runner doesn't need custom
replacement/expansion logic.
> Don't attempt to fuse subtransforms of primitive/known transforms.
> ------------------------------------------------------------------
>
> Key: BEAM-6327
> URL: https://issues.apache.org/jira/browse/BEAM-6327
> Project: Beam
> Issue Type: New Feature
> Components: runner-direct
> Reporter: Robert Bradshaw
> Assignee: Kyle Weaver
> Priority: Major
> Labels: portability
> Time Spent: 2h 20m
> Remaining Estimate: 0h
>
> Currently we must remove all sub-components of any known transform that may
> have an optional substructure, e.g.
> [https://github.com/apache/beam/blob/release-2.9.0/sdks/python/apache_beam/runners/portability/portable_runner.py#L126]
> (for GBK) and [https://github.com/apache/beam/pull/7360] (Reshuffle).
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
