You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Davor Bonaci (JIRA)" <ji...@apache.org> on 2016/11/11 22:30:58 UTC
[jira] [Commented] (BEAM-961) CountingInput could have starting
number
[ https://issues.apache.org/jira/browse/BEAM-961?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15658361#comment-15658361 ]
Davor Bonaci commented on BEAM-961:
-----------------------------------
Just keeping the JIRA updated:
On Thu, Nov 10, 2016 at 2:02 PM, Kenneth Knowles <kl...@google.com.invalid> wrote:
I'm not particular about whether the source itself does it versus replacing
uses of the source with a new PTransform encapsulating them. As long as
there is some object with `startingAt` and `upTo`. Either way should be
easy? Left it with starter tag as it would potentially be a fun initial
dive into the codebase.
On Thu, Nov 10, 2016 at 1:23 PM, Dan Halperin <dh...@google.com> wrote:
> Why not support this in a follow-on pardo that shifts the range?
> CountingInput could have starting number
> ----------------------------------------
>
> Key: BEAM-961
> URL: https://issues.apache.org/jira/browse/BEAM-961
> Project: Beam
> Issue Type: New Feature
> Components: sdk-java-core
> Reporter: Kenneth Knowles
> Priority: Trivial
> Labels: easy, newbie, starter
>
> TL;DR: Add {{startingAt}} to {{CountingInput}}.
> Right now you can have {{CountingInput.upTo(someNumber)}} but it came up in a test that if you want to have, say, one PCollection that is 1 through 10 and another that is 11 through 20 - so you know they are disjoint - then it requires some boilerplate to add 10 to every element. That boilerplate should be part of the {{CountingInput}}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)