[BEAM-7956] FloatCoder and DoubleCoder structuralValue doesn't handle NaN

• Type: Bug
• Status: Resolved
• Priority: P4
• Resolution: Invalid
• Affects Version/s: 2.14.0
• Fix Version/s: Not applicable
• Component/s: sdk-java-core
• Labels: Done
• Assignee: Unassigned
• Reporter: Neville Li
• Created: 12/Aug/19 15:23
• Updated: 06/Oct/20 21:50
• Resolved: 13/Aug/19 14:29

Description

Comparing Float and Double NaN values always yields false, even though they serialize consistently to the same byte array.
FloatCoder and DoubleCoder override consistentWithEquals to true but doesn't override structuralValue which return the orignal NaN and fails comparison.

This trips MutationDetectors in tests.