Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-11637

BitAnd unsafely mutates an instance field.

Details

    • Bug
    • Status: Resolved
    • P2
    • Resolution: Fixed
    • None
    • 2.28.0
    • dsl-sql
    • None

    Description

      BitAnd uses isEmpty, an instance field, to track if inputs seen thus far are null. This is an unsafe usage of CombineFn, since all mutable state is expected to be stored in the Accumulator.

      Attachments

        Issue Links

          Activity

            People

              ibzib Kyle Weaver
              ibzib Kyle Weaver
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1h 10m
                  1h 10m