You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@beam.apache.org by "Kyle Weaver (Jira)" <ji...@apache.org> on 2020/05/04 18:50:00 UTC

[jira] [Created] (BEAM-9878) Update ElementByteSizeObservableIterable

Kyle Weaver created BEAM-9878:
---------------------------------

             Summary: Update ElementByteSizeObservableIterable
                 Key: BEAM-9878
                 URL: https://issues.apache.org/jira/browse/BEAM-9878
             Project: Beam
          Issue Type: Improvement
          Components: runner-dataflow
            Reporter: Kyle Weaver


It's odd that ElementByteSizeObservableIterable::iterator adds observers within the method body. I assume this is for historic reasons, since it doesn't seem to do anything now, and the comment documenting references a setObserver method that doesn't exist.

https://github.com/apache/beam/blob/6453e859badcb629ae2528b77d84235b7291ff89/sdks/java/core/src/main/java/org/apache/beam/sdk/util/common/ElementByteSizeObservableIterable.java#L49-L61



--
This message was sent by Atlassian Jira
(v8.3.4#803005)