weiqingy commented on PR #28692: URL: https://github.com/apache/flink/pull/28692#issuecomment-5160767517
Closing this in favor of a smaller PR series. FLIP-485 passed the vote on 2026-08-01 with 3 binding +1 and no -1: https://lists.apache.org/thread/symqpswsohl2s5wmtkcw0jjp1w5dot0n This draft was opened as a single reference implementation so the DISCUSS and VOTE threads had something concrete to read against master. Now that the design is settled, the same code is going up as four smaller, independently reviewable PRs: 1. `UdfMetrics` helper: https://github.com/apache/flink/pull/28878 2. Config options plus synchronous scalar and table instrumentation 3. Asynchronous scalar and table instrumentation 4. Documentation The code is unchanged from what was reviewed here. It is only partitioned and rebased onto current master. @davidradl thanks again for the overhead question earlier. The note that the sampling keeps the enabled-path cost down goes into the documentation PR, number 4 in the list above. -- 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]
