[
https://issues.apache.org/jira/browse/BEAM-3926?focusedWorklogId=104878&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-104878
]
ASF GitHub Bot logged work on BEAM-3926:
----------------------------------------
Author: ASF GitHub Bot
Created on: 22/May/18 23:54
Start Date: 22/May/18 23:54
Worklog Time Spent: 10m
Work Description: robertwb commented on a change in pull request #5437:
[BEAM-3926] Add new metrics protos based on "Defining and adding SDK Metrics"
htt…
URL: https://github.com/apache/beam/pull/5437#discussion_r190083759
##########
File path: model/fn-execution/src/main/proto/beam_fn_api.proto
##########
@@ -239,13 +239,18 @@ message ProcessBundleRequest {
// Stable
message ProcessBundleResponse {
- // (Optional) If metrics reporting is supported by the SDK, this represents
- // the final metrics to record for this bundle.
- Metrics metrics = 1;
+ // DEPRECATED. Please use monitoring_info.
+ Metrics metrics = 1 [deprecated=true];
// (Optional) Specifies that the bundle has been split since the last
// ProcessBundleProgressResponse was sent.
BundleSplit split = 2;
+
+ // (Optional) If metrics or monitored sttate reporting is supported by
Review comment:
s/sttate/state
----------------------------------------------------------------
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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 104878)
Time Spent: 50m (was: 40m)
> Support MetricsPusher in Dataflow Runner
> ----------------------------------------
>
> Key: BEAM-3926
> URL: https://issues.apache.org/jira/browse/BEAM-3926
> Project: Beam
> Issue Type: Sub-task
> Components: runner-dataflow
> Reporter: Scott Wegner
> Assignee: Pablo Estrada
> Priority: Major
> Time Spent: 50m
> Remaining Estimate: 0h
>
> See [relevant email
> thread|https://lists.apache.org/thread.html/2e87f0adcdf8d42317765f298e3e6fdba72917a72d4a12e71e67e4b5@%3Cdev.beam.apache.org%3E].
> From [~echauchot]:
>
> _AFAIK Dataflow being a cloud hosted engine, the related runner is very
> different from the others. It just submits a job to the cloud hosted engine.
> So, no access to metrics container etc... from the runner. So I think that
> the MetricsPusher (component responsible for merging metrics and pushing them
> to a sink backend) must not be instanciated in DataflowRunner otherwise it
> would be more a client (driver) piece of code and we will lose all the
> interest of being close to the execution engine (among other things
> instrumentation of the execution of the pipelines). I think that the
> MetricsPusher needs to be instanciated in the actual Dataflow engine._
>
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
