lukecwik commented on a change in pull request #11177: [BEAM-9562] Add Timer to 
Elements proto representation.
URL: https://github.com/apache/beam/pull/11177#discussion_r395426028
 
 

 ##########
 File path: model/fn-execution/src/main/proto/beam_fn_api.proto
 ##########
 @@ -164,6 +164,11 @@ message ProcessBundleDescriptor {
   // data plane or if any of the transforms rely on user state or side inputs.
   org.apache.beam.model.pipeline.v1.ApiServiceDescriptor
       state_api_service_descriptor = 7;
+
+  // A descriptor describing the end  point to use for Data API to use user
+  // timers.
 
 Review comment:
   ```suggestion
     // A descriptor describing the end point to use for Data API for user 
timers.
     // Required if the ProcessBundleDescriptor contains any transforms that 
have user timers.
   ```

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


With regards,
Apache Git Services

Reply via email to