JayajP commented on PR #30089: URL: https://github.com/apache/beam/pull/30089#issuecomment-1908673959
Thanks for the testing tips. Two points of note - Looks like there isn't a standard `TestClock` so I created a barebones class for in the unit test. - I also needed to mark the member function `StreamingStepMetricsContainer::extractPerWorkerMetrics` as `@VisibleForTesting`, otherwise I would also need to make modifications to the `MetricsContainerRegistry` which seems unnecessarily complex. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
