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

Rename one of PTransform.apply and PInput.apply

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

      Description

      Before releasing Beam 1.0, we should do this.

      Right now, it's legal to call:

      ptransform.apply(input)

      and

      input.apply(ptransform)

      when only the latter is correct. The former skips various validation steps and loses the notion of composite transforms.

        Attachments

          Activity

            People

            • Assignee:
              kenn Kenneth Knowles
              Reporter:
              dhalperi Dan Halperin

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment