[jira] [Assigned] (BEAM-8017) Plumb errors and remove panics from package graphx

["Jing Chen (Jira)" <ji...@apache.org>]

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

Jing Chen reassigned BEAM-8017:
-------------------------------

    Assignee: Jing Chen

> Plumb errors and remove panics from package graphx
> --------------------------------------------------
>
>                 Key: BEAM-8017
>                 URL: https://issues.apache.org/jira/browse/BEAM-8017
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-go
>            Reporter: Robert Burke
>            Assignee: Jing Chen
>            Priority: P3
>              Labels: Novice, beginner, noob, starter
>
> The graphx package, and in particular serialize.go and coder.go should be returning errors back up, rather than panicing when issues occur deeper when marshalling types. It makes errors harder to follow since there's now a less necessary panic trace to skip, rather than a clearly constructed error message.
> Not difficult, but may be tedious. Requires plumbing the errors and handling/wrapping them appropriately instead of using panic. Most error handling is presently correctly wrapped anyway.
> The graphx package as a rule is intended for beam internal use, and not part of the user surface, so making the API changes (which aren't backwards compatible) isn't the worst. Most of the affected methods are unexported.



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