[
https://issues.apache.org/jira/browse/BEAM-6652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16769744#comment-16769744
]
Robert Burke commented on BEAM-6652:
------------------------------------
Looking into it more deeply, I discovered that the Go Standard Library JSON
encoder documents that maps are encoded deterministically:
[https://github.com/golang/go/blob/master/src/encoding/json/encode.go#L135]
I will add a test to verify this behavior, and add a tester for coders to
ensure they're deterministic.
> Handle map encoding better.
> ---------------------------
>
> Key: BEAM-6652
> URL: https://issues.apache.org/jira/browse/BEAM-6652
> Project: Beam
> Issue Type: Improvement
> Components: sdk-go
> Reporter: Robert Burke
> Assignee: Robert Burke
> Priority: Minor
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> Maps are a dangerous type to have as part of key values, and the Go SDK
> currently doesn't handle them very well. Until we have a default coder that
> assures that map encodings are canonical, it's better that we restrict their
> usage within the SDK.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
