mateczagany commented on code in PR #26348:
URL: https://github.com/apache/flink/pull/26348#discussion_r2014429040
##########
flink-metrics/flink-metrics-otel/pom.xml:
##########
@@ -143,4 +143,39 @@ under the License.
</dependency>
</dependencies>
+
+ <build>
Review Comment:
I think the best solution would be to introduce a new e2e test for
OpenTelemetry, like the currently existing Prometheus metric reporter test.
That would catch issues like this in the future.
After that, we could more confidently increase the version of the
OpenTelemetry package too, as 1.30.1 is from 2023, and the newest one would be
1.48.0.
I will create JIRAs for these tasks.
--
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]