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

Outdated Transform.Inputs values when chaining expanded transforms

Details

    • Bug
    • Status: Resolved
    • P2
    • Resolution: Fixed
    • None
    • Missing
    • sdk-go
    • None

    Description

      For:
        pcol := ...
        tmp := beam.CrossLanguage(..., {"foo": pcol}, ...)
        out := beam.CrossLanguage(..., {"bar": tmp}, ...)

      In the result model pipeline, the second expanded transform's inputs may hold a map from local name to the original PCollection/node ID, even if the first expansion has changed the output IDs.

      Looks like a fix is to move the output name remapping after mergeExpandedWithPipeline: https://github.com/apache/beam/blob/928da54e1a95219bfbfa420fbf9c8fb5514fd683/sdks/go/pkg/beam/core/runtime/graphx/translate.go#L160-L163

      Attachments

        Issue Links

          Activity

            People

              htyleo Tianyang Hu
              htyleo Tianyang Hu
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1h 40m
                  1h 40m