[
https://issues.apache.org/jira/browse/BEAM-8017?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Chen updated BEAM-8017:
----------------------------
Status: Resolved (was: Open)
> 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
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
> 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)
