[jira] [Updated] (BEAM-9982) Replace graphx.MustMarshal with protox.MustEncode

["Kenneth Knowles (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/BEAM-9982?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kenneth Knowles updated BEAM-9982:
----------------------------------
    Resolution: Fixed
        Status: Resolved  (was: Resolved)

Hello! Due to a bug in our Jira configuration, this issue had status:Resolved but resolution:Unresolved.

I am bulk editing these issues to have resolution:Fixed

If a different resolution is appropriate, please change it. To do this, click the "Resolve" button (you can do this even for closed issues) and set the Resolution field to the right value.

> Replace  graphx.MustMarshal with protox.MustEncode
> --------------------------------------------------
>
>                 Key: BEAM-9982
>                 URL: https://issues.apache.org/jira/browse/BEAM-9982
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-go
>            Reporter: Robert Burke
>            Assignee: Robert Burke
>            Priority: P3
>             Fix For: Not applicable
>
>
> A redundant helper function, [graphx.MustMarshal|https://github.com/apache/beam/blob/master/sdks/go/pkg/beam/core/runtime/graphx/translate.go#L117], was accidentally introduced recently. There exists an identical function in a different package that was already being used in that same file, [protox.MustEncode|https://github.com/apache/beam/blob/master/sdks/go/pkg/beam/core/util/protox/protox.go#L22]
> This task is to remove all instances of the graphx.MustMarshal function and replace them with the protox.MustEncode call instead.



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