ASF GitHub Bot logged work on BEAM-3741:

                Author: ASF GitHub Bot
            Created on: 12/Mar/18 03:11
            Start Date: 12/Mar/18 03:11
    Worklog Time Spent: 10m 
      Work Description: jkff commented on a change in pull request #4743: 
[BEAM-3741] Proto changes for splitting over Fn API
URL: https://github.com/apache/beam/pull/4743#discussion_r173687202

 File path: model/fn-execution/src/main/proto/beam_fn_api.proto
 @@ -199,6 +232,10 @@ message ProcessBundleResponse {
   // (Optional) If metrics reporting is supported by the SDK, this represents
   // the final metrics to record for this bundle.
   Metrics metrics = 1;
+  // (Optional) Specifies that the bundle has been split since the last
+  // ProcessBundleProgressResponse was sent.
+  BundleSplit split = 2;
 Review comment:
   I agree that https://issues.apache.org/jira/browse/BEAM-3787 is the right 
way to go, however I'd like to make progress on the current PR and followup 
work and have some protos upon which to build the SDK harness and runner 
harness changes - I think as BEAM-3787 is implemented, it will be fairly easy 
to port onto that new API; I'm willing to do it for splitting in particular 
when it's time.

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

Issue Time Tracking

    Worklog Id:     (was: 79325)
    Time Spent: 3h 50m  (was: 3h 40m)

> Proto changes for splitting over Fn API
> ---------------------------------------
>                 Key: BEAM-3741
>                 URL: https://issues.apache.org/jira/browse/BEAM-3741
>             Project: Beam
>          Issue Type: Sub-task
>          Components: beam-model
>            Reporter: Eugene Kirpichov
>            Assignee: Eugene Kirpichov
>            Priority: Major
>          Time Spent: 3h 50m
>  Remaining Estimate: 0h

This message was sent by Atlassian JIRA

Reply via email to