Hi Prasanna, Why do you need histograms vs summaries? I'm curious about the change and want to see if it applies to my usage of the PrometheusReporter.
Best, Mason On Mon, Jan 31, 2022 at 11:51 PM Martijn Visser <mart...@ververica.com> wrote: > Hi Prasanna, > > Just a quick note that the Github links are all pointing to release > candidate 1 for 1.14.3. The released version is in > https://github.com/apache/flink/blob/release-1.14.3/flink-runtime/src/test/java/org/apache/flink/runtime/metrics/MetricRegistryTestUtils.java > > Best regards, > > Martijn > > On Tue, 1 Feb 2022 at 07:35, Prasanna kumar <prasannakumarram...@gmail.com> > wrote: > >> NVM was able to find it in a different place >> >> >> https://github.com/apache/flink/blob/release-1.14.3-rc1/flink-runtime/src/test/java/org/apache/flink/runtime/metrics/MetricRegistryTestUtils.java >> >> On Tue, Feb 1, 2022 at 11:58 AM Prasanna kumar < >> prasannakumarram...@gmail.com> wrote: >> >>> Hi, >>> >>> Team, We are writing our own prometheus reporter to make sure that we >>> are capturing data in histograms rather than summaries. >>> >>> We were able to do it successfully in version 1.12.7. >>> >>> But while upgrading to version 1.14.3 , we find >>> that MetricRegistryTestUtils is not available in the src code given by >>> flink in github. >>> >>> PrometheusReporterTest.java throws error that this file is unavailable >>> >>> [image: Screen Shot 2022-02-01 at 11.50.09 AM.png] >>> >>> [image: Screen Shot 2022-02-01 at 11.53.17 AM.png] >>> >>> The below code is in 1.12.7 but method >>> defaultMetricRegistryConfiguration is deprecated in the latest version. >>> >>> [image: Screen Shot 2022-02-01 at 11.51.46 AM.png] >>> >>> Looking at the Github link >>> https://github.com/apache/flink/tree/release-1.14.3-rc1/flink-runtime/src/main/java/org/apache/flink/runtime/metrics >>> also shows that the MetricRegistryTestUtils is not available. It's not >>> available in >>> https://github.com/apache/flink/tree/master/flink-runtime/src/main/java/org/apache/flink/runtime/metrics >>> master branch as well. >>> >>> [image: Screen Shot 2022-02-01 at 11.55.19 AM.png] >>> >>> Could the community please add the Class file in GITHUB. >>> >>> Thanks, >>> Prasanna. >>> >>
