[BEAM-7269] Remove StateSpec from hashCode of SimpleStateTag

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Affects Version/s: 2.12.0
• Fix Version/s: 2.14.0
• Component/s: runner-core
• Assignee: Jan Lukavsky
• Reporter: Jan Lukavský
• Created: 10/May/19 20:21
• Updated: 24/Jul/20 19:54
• Resolved: 13/May/19 16:11

Description

SimpleStateTag is used as key in hash based StateTable, and currently is hashes and compares StateSpec and StructuredId, that are inside the SimpleStateTag. StateSpec hashes Coder into the resulting hashCode and when the Coder fails to have a proper hashCode and equals implementations, it results in wrong behavior (apparently missing states).

Issue Links

• relates to: BEAM-6813 Issues with state + timers in java Direct Runner (state cell is null)
• links to: GitHub Pull Request #8555