Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-10779

Support multiple main inputs and side inputs for PTransformOverride

    XMLWordPrintableJSON

    Details

    • Type: New Feature
    • Status: Open
    • Priority: P2
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: sdk-py-core
    • Labels:
      None

      Description

      Currently, PTransformOverride only supports overriding PTransform s that have a single input. It would be useful to support multiple main inputs and side inputs as well.

      Proposal:

      • Introduce a new method PTransformOverride.get_replacement_inputs(self, applied_ptransform) that can be overridden by end users. This method will be called to get the inputs for the replacement PTransform.
      • The default implementation will return a tuple of all the main inputs in order, concatenated with the all the side inputs PValue s in order.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                myffical@gmail.com Yifan Mai
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 8.5h
                  8.5h