You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/04 12:57:33 UTC
[GitHub] [beam] damccorm opened a new issue, #19735: Detect unexported fields in unregistered types for better error messages
damccorm opened a new issue, #19735:
URL: https://github.com/apache/beam/issues/19735
Unregistered types with unexported fields cause pipeline execution time panics rather than construction time panics/errors, which would be preferable.
Improved error messages encouraging users to register their types already exists, but isn't currently triggered in this instance, so it's a matter of returning an error in this case.
eg. for using time.Time as a value in a DoFn...
panic:
<Coroner crash reporting disabled due to unit tests.\>
reflect.StructOf: StructOf does not allow unexported fields [recovered]
panic: reflect.StructOf: StructOf does not allow unexported fields
goroutine 195 [running]:
panic(0x7e0a060, 0x84beb70)
third_party/go/gc/src/runtime/panic.go:567 ****0x2da fp=0xc000dc1178 sp=0xc000dc10c0 pc=0xee24daa
testing.tRunner.func1(0xc000d6ec00)
third_party/go/gc/src/testing/testing.go:830 ****0x388 fp=0xc000dc11f8 sp=0xc000dc1178 pc=0xfafda58
runtime.call32(0x0, 0x833e1e0, 0xc000caaab0, 0x800000008)
third_party/go/gc/src/runtime/asm_amd64.s:519 ****0x3b fp=0xc000dc1228 sp=0xc000dc11f8 pc=0xee53acb
panic(0x7e0a060, 0x84beb70)
third_party/go/gc/src/runtime/panic.go:522 ****0x1b5 fp=0xc000dc12e0 sp=0xc000dc1228 pc=0xee24c85
reflect.runtimeStructField(0xc000e267e0, 0x4, 0xc000e267e4, 0x4, 0x8541880, 0x7e0a060, 0x0, 0x0, 0x0, 0xc000e26990, ...)
third_party/go/gc/src/reflect/type.go:2765 ****0x1c2 fp=0xc000dc1348 sp=0xc000dc12e0 pc=0xee80ea2
reflect.StructOf(0xc0006dd040, 0x3, 0x4, 0x0, 0x0)
third_party/go/gc/src/reflect/type.go:2371 ****0x21f6 fp=0xc000dc1b28 sp=0xc000dc1348 pc=0xee7f816
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48800](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48800), 0xc000dc1e68, 0xee3c4f2, 0xc000e26920, 0x0)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:558 ****0xa6c fp=0xc000dc1df8 sp=0xc000dc1b28 pc=0xf53a7ac
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48780](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48780), 0x0, 0x0, 0x0, 0x1)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:533 ****0x471 fp=0xc000dc20c8 sp=0xc000dc1df8 pc=0xf53a1b1
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48680](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48680), 0xc000369bf0, 0xc000b24a40, 0x3e, 0xc000961728)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:542 ****0x6c5 fp=0xc000dc2398 sp=0xc000dc20c8 pc=0xf53a405
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48600](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48600), 0xc0001af340, 0x1, 0x1, 0x2)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:583 ****0x12fc fp=0xc000dc2668 sp=0xc000dc2398 pc=0xf53b03c
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48480](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48480), 0xc0001af2d0, 0x1, 0x1, 0x2)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:542 ****0x6c5 fp=0xc000dc2938 sp=0xc000dc2668 pc=0xf53a405
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48280](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48280), 0x20, 0xc000e265c0, 0x0, 0x1)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:542 ****0x6c5 fp=0xc000dc2c08 sp=0xc000dc2938 pc=0xf53a405
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48200](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48200), 0x0, 0x0, 0x0, 0x1)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:533 ****0x471 fp=0xc000dc2ed8 sp=0xc000dc2c08 pc=0xf53a1b1
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48080](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeType(0xc000c48080), 0x0, 0xc000dc3230, 0xef9eb65, 0xc000dc3208)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:542 ****0x6c5 fp=0xc000dc31a8 sp=0xc000dc2ed8 pc=0xf53a405
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeCustomCoder(0xc000cc2750](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.decodeCustomCoder(0xc000cc2750), 0x698, 0x8512f20, 0xc000cc2750)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/serialize.go:146 ****0x44 fp=0xc000dc32b0 sp=0xc000dc31a8 pc=0xf535e14
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.(*CoderUnmarshaller).makeCoder(0xc0003250e0](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.(*CoderUnmarshaller).makeCoder(0xc0003250e0), 0xc000985500, 0xc000ec7c18, 0x2, 0xc000dcdb10)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/coder.go:227 ****0x1bab fp=0xc000dc35a0 sp=0xc000dc32b0 pc=0xf52eefb
[google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.(*CoderUnmarshaller).Coder(0xc0003250e0](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/graphx.(*CoderUnmarshaller).Coder(0xc0003250e0), 0xc000ec7c18, 0x2, 0xc0000511c8, 0xc000dc3748, 0xc000051da0)
third_party/golang/apache_beam/pkg/beam/core/runtime/graphx/coder.go:111 ****0xe0 fp=0xc000dc3650 sp=0xc000dc35a0 pc=0xf52ccc0
[google3/third_party/golang/apache_beam/pkg/beam/runners/dataflow/dataflowlib/dataflowlib.(*translator).translateCoder(0xc000325100](https://cs.corp.google.com/piper///depot/google3/third_party/golang/apache_beam/pkg/beam/runners/dataflow/dataflowlib/dataflowlib.(*translator).translateCoder(0xc000325100), 0xc000eec660, 0xc000ec7c18, 0x2, 0xc000051e48)
Imported from Jira [BEAM-8018](https://issues.apache.org/jira/browse/BEAM-8018). Original Jira may contain additional context.
Reported by: lostluck.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org