mateczagany commented on code in PR #26348:
URL: https://github.com/apache/flink/pull/26348#discussion_r2012484224
##########
flink-metrics/flink-metrics-otel/pom.xml:
##########
@@ -143,4 +143,39 @@ under the License.
</dependency>
</dependencies>
+
+ <build>
Review Comment:
Hi David, thank you for your comment.
Unfortunately I don't see a way to add a minimal unit test as this was a
dependency packaging error. We would need to have an E2E test for this to come
up during CI, as that's the only way I see that we could test the final
packaged JAR.
I would rather not do that in this PR, but we can open another issue for
that if you want.
--
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]
