You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@beam.apache.org by "Beam JIRA Bot (Jira)" <ji...@apache.org> on 2020/08/01 17:07:04 UTC
[jira] [Commented] (BEAM-10170) TextBasedReader may not respect the
source end offset
[ https://issues.apache.org/jira/browse/BEAM-10170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17169328#comment-17169328 ]
Beam JIRA Bot commented on BEAM-10170:
--------------------------------------
This issue is P2 but has been unassigned without any comment for 60 days so it has been labeled "stale-P2". If this issue is still affecting you, we care! Please comment and remove the label. Otherwise, in 14 days the issue will be moved to P3.
Please see https://beam.apache.org/contribute/jira-priorities/ for a detailed explanation of what these priorities mean.
> TextBasedReader may not respect the source end offset
> -----------------------------------------------------
>
> Key: BEAM-10170
> URL: https://issues.apache.org/jira/browse/BEAM-10170
> Project: Beam
> Issue Type: Bug
> Components: beam-model
> Reporter: Jessica Wise
> Priority: P2
> Labels: stale-P2
>
> [TextBasedReader|https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/io/TextSource.java#L88] is backed by a TextSource, which may have a start and end offset. If the end offset does not correspond to a delimiter, the TextBasedReader will not respect the end offset and will instead read past the end offset to the next instance of a delimiter. See [TextBasedReader#findDelimiterBounds|https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/io/TextSource.java#L176] which finds the end of the next record to read: this method will "consume the channel till either EOF or the delimiter bounds are found." I believe this is a bug and that this method should also check for the end offset, not just EOF or a delimiter.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)