Skip to content

[exporter/doris] Second PR (with metrics Implementation) of New component: Doris Exporter #35338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Oct 30, 2024

Conversation

joker-star-l
Copy link
Contributor

Description:

Second PR of New component: Doris Exporter. Implementation of metrics.

Link to tracking Issue: #33479

Testing:

Tested via unit test.

Documentation:

No additional documentation.

Copy link
Contributor

github-actions bot commented Oct 7, 2024

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@joker-star-l
Copy link
Contributor Author

@atoulme Hello, Is there any problem with this PR?

@atoulme
Copy link
Contributor

atoulme commented Oct 18, 2024

Maybe this can use more unit test coverage, but overall looks ok for a first push.

@joker-star-l
Copy link
Contributor Author

Maybe this can use more unit test coverage, but overall looks ok for a first push.

Ok. Thank you.

@joker-star-l
Copy link
Contributor Author

Could someone help me merge this pr?

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Oct 28, 2024
@joker-star-l
Copy link
Contributor Author

Could someone help me merge this pr? Sorry to ask again.

@andrzej-stencel andrzej-stencel merged commit 5f3ff6c into open-telemetry:main Oct 30, 2024
168 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 30, 2024
zzhlogin pushed a commit to zzhlogin/opentelemetry-collector-contrib-aws that referenced this pull request Nov 12, 2024
…nent: Doris Exporter (open-telemetry#35338)

**Description:** <Describe what has changed.>

Second PR of New component: Doris Exporter. Implementation of metrics.

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->

**Link to tracking Issue:** open-telemetry#33479 

**Testing:** 

Tested via unit test.

**Documentation:** <Describe the documentation added.>

No additional documentation.
@joker-star-l joker-star-l deleted the metrics branch December 5, 2024 15:23
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
…nent: Doris Exporter (open-telemetry#35338)

**Description:** <Describe what has changed.>

Second PR of New component: Doris Exporter. Implementation of metrics.

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->

**Link to tracking Issue:** open-telemetry#33479 

**Testing:** 

Tested via unit test.

**Documentation:** <Describe the documentation added.>

No additional documentation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/doris ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants