[
https://issues.apache.org/jira/browse/BEAM-4667?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17547296#comment-17547296
]
Kenneth Knowles commented on BEAM-4667:
---------------------------------------
This issue has been migrated to https://github.com/apache/beam/issues/19064
> Potential issue with QuantileStateCoder
> ---------------------------------------
>
> Key: BEAM-4667
> URL: https://issues.apache.org/jira/browse/BEAM-4667
> Project: Beam
> Issue Type: Bug
> Components: sdk-java-core
> Reporter: Zhiheng Huang
> Priority: P3
> Labels: Clarified
>
> [https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/ApproximateQuantiles.java#L687]
> The line above encodes the QuantileState buffers.size() as if it's
> numBuffers. This seems wrong since before buffers are full, buffers.size() is
> not equal to numBuffers. One thing I suspect will happen is that, if we
> serialize before buffer is full, it will effectively reduce the number of
> buffers we maintain after deserialization.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
