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 2021/03/31 18:27:00 UTC

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

     [ https://issues.apache.org/jira/browse/BEAM-9878?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kyle Weaver updated BEAM-9878:
------------------------------
    Component/s:     (was: runner-dataflow)
                 sdk-java-core

> Update ElementByteSizeObservableIterable
> ----------------------------------------
>
>                 Key: BEAM-9878
>                 URL: https://issues.apache.org/jira/browse/BEAM-9878
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>            Reporter: Kyle Weaver
>            Priority: P3
>
> 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)