[BEAM-14166] Improvements to RowWithGetter

• Type: Improvement
• Status: Open
• Priority: P2
• Resolution: Unresolved
• Component/s: sdk-java-core
• Assignee: Moritz Mack
• Reporter: Moritz Mack
• Created: 24/Mar/22 13:37
• Updated: 05/Jun/22 00:35

Description

Various improvements to getValue(fieldIdx) in RowWithGetters mentioned here

• Minimize memory overhead of cache using either a index lookup (array) or a single hash map if number of fields exceeds the initial hashmap capacity
• The cache should be checked before calling a getter to avoid any potentially unnecessary conversion in the getter itself.
• Nested rows should be cached, otherwise the cache of such nested rows can't be leveraged.

• Handling of collections / maps / iterables can be significantly improved by simply skipping the transform in all cases where getValue for members is the identity transform. 

Issue Links

• links to: GitHub Pull Request #17172