[BEAM-14093] OffsetRestrictionTracker calculates wrongly the tryClaim conditions

• Type: Bug
• Status: Resolved
• Priority: P1
• Resolution: Invalid
• Affects Version/s: 2.37.0
• Fix Version/s: Not applicable
• Component/s: io-py-common
• Assignee: Israel Herraiz
• Reporter: Israel Herraiz
• Created: 13/Mar/22 21:59
• Updated: 13/Mar/22 22:06
• Resolved: 13/Mar/22 22:06

Description

The OffsetRestrictionTracker method tryClaim tries to calculate if the claim can be done using this condition:

if self._range.start <= position < self._range.stop:
  self._current_position = position
  return True

The condition should be

if self._range.start <= position and position < self._range.stop:
  self._current_position = position
  return True

This affects the correctness of the claims for SplittableDoFns.