anishmehta24 commented on issue #38746: URL: https://github.com/apache/beam/issues/38746#issuecomment-5721870540
Since the first attempt (#38749) was reverted in #38901 and has been idle since June, I've prepared a re-land that keeps `DelegatingCounter.inc` / `DelegatingStringSet.add` / `DelegatingBoundedTrie.add` as `MetricUpdater` attributes (so `inc(value=...)` and pickling behave exactly as before) and gates the update inside `MetricUpdater.__call__` instead, mirroring how the Java SDK checks `MetricsFlag` on every call. @Anuragp22 happy to hand it back if you're still on this — otherwise PR incoming. -- 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]
