ajamato commented on pull request #14490: URL: https://github.com/apache/beam/pull/14490#issuecomment-840272513
i have split this up into two PRs, rather than 3. As the startTime logic is much simpler now, and only done for the process wide MetricContainer. Overall the whole thing is much smaller. PTAL (1) Change the way ShortIdMap works, https://github.com/apache/beam/pull/14804 (2) Add process-wide metrics, and add the notion of startTime to a subset of metrics. https://github.com/apache/beam/pull/14805 -- 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: [email protected]
