[BEAM-14177] GroupByKey iteration caching broken for portable runners like Dataflow runner v2

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: 2.38.0
• Component/s: sdk-java-harness
• Assignee: Luke Cwik
• Reporter: Luke Cwik
• Created: 25/Mar/22 16:33
• Updated: 29/Mar/22 20:20
• Resolved: 29/Mar/22 20:20

Description

The wrong cache key is being used as it has not been namespaced to the state key.

This was previously being done within StateFetchingIterators but https://github.com/apache/beam/pull/17121 changed that to use a single shared key.

The fix is to subcache the cache before passing it into StateFetchingIterators restoring the prior behavior.

Issue Links

• links to: GitHub Pull Request #17184
• links to: GitHub Pull Request #17193