Skip to content

Conversation

@michaelkoepf
Copy link
Contributor

@michaelkoepf michaelkoepf commented Jun 24, 2025

Purpose

Linked issue: close #1268

Brief change log

Add OpenTelemetry metric exporter (fluss-metrics-opentelemetry)

Tests

Unit Tests

  • fluss-metrics/fluss-metrics-opentelemetry/src/test/java/com/alibaba/fluss/metrics/opentelemetry/MetricMetadataTest.java
  • fluss-metrics/fluss-metrics-opentelemetry/src/test/java/com/alibaba/fluss/metrics/opentelemetry/OpenTelemetryAdapterTest.java
  • fluss-metrics/fluss-metrics-opentelemetry/src/test/java/com/alibaba/fluss/metrics/opentelemetry/OpenTelemetryReporterPluginTest.java
  • fluss-metrics/fluss-metrics-opentelemetry/src/test/java/com/alibaba/fluss/metrics/opentelemetry/OpenTelemetryReporterTest.java

Integration Tests

Note: This PR also introduces new test dependencies (testcontainers) and test utilities (fluss-test-utils/src/main/java/com/alibaba/fluss/testutils/common/TestContainerExtension.java) for integration tests in fluss-test-utils.

  • fluss-metrics/fluss-metrics-opentelemetry/src/test/java/com/alibaba/fluss/metrics/opentelemetry/OpenTelemetryReporterITCase.java

API and Format

n/a

Documentation

See #1269

@michaelkoepf michaelkoepf force-pushed the metrics/opentelemetry-metrics-exporter branch from e3438cf to 53e01b9 Compare June 24, 2025 15:51
@michaelkoepf michaelkoepf marked this pull request as ready for review June 24, 2025 15:52
@michaelkoepf
Copy link
Contributor Author

Note: Rerun CI pipeline when Docker is installed on runners.

@michaelkoepf michaelkoepf force-pushed the metrics/opentelemetry-metrics-exporter branch 3 times, most recently from bbaee94 to 1462ca4 Compare June 30, 2025 22:10
@luoyuxia
Copy link
Contributor

luoyuxia commented Jul 4, 2025

@michaelkoepf Hi, could you please divide this pr into two pull requests, one for code, another for docs? I'll help review the code part firstly. Currenty, it looks a little big to me.

@michaelkoepf michaelkoepf marked this pull request as draft July 4, 2025 07:31
@michaelkoepf michaelkoepf force-pushed the metrics/opentelemetry-metrics-exporter branch from 1462ca4 to 348ebe0 Compare July 4, 2025 07:51
@michaelkoepf
Copy link
Contributor Author

@luoyuxia done. implementation is committed in this pr, docs and quickstart in #1270

@michaelkoepf michaelkoepf marked this pull request as ready for review July 4, 2025 08:37
@michaelkoepf michaelkoepf force-pushed the metrics/opentelemetry-metrics-exporter branch 2 times, most recently from d57f807 to 351dcff Compare August 29, 2025 18:32
@michaelkoepf michaelkoepf marked this pull request as draft August 29, 2025 19:31
michaelkoepf added a commit to michaelkoepf/fluss that referenced this pull request Aug 29, 2025
@michaelkoepf michaelkoepf marked this pull request as ready for review August 29, 2025 19:56
@polyzos polyzos force-pushed the main branch 3 times, most recently from d88c76c to 434a4f4 Compare August 31, 2025 15:13
@michaelkoepf michaelkoepf force-pushed the metrics/opentelemetry-metrics-exporter branch from 57030dd to c758d27 Compare September 26, 2025 07:36
michaelkoepf added a commit to michaelkoepf/fluss that referenced this pull request Sep 26, 2025
michaelkoepf added a commit to michaelkoepf/fluss that referenced this pull request Nov 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[metrics] OpenTelemetry Metrics Exporter Implementation

2 participants