Skip to content
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

[ingester] support multiple OTLP exporters #4230 #4177

Merged
merged 2 commits into from
Sep 8, 2023

Conversation

lzf575
Copy link
Contributor

@lzf575 lzf575 commented Sep 7, 2023

This PR is for:

  • Server

@jiekun
Copy link
Contributor

jiekun commented Sep 7, 2023

I'm having a few suggestion in related issue: #4157 (comment) , and claim that it won't solve the requirement that sending data to different platform because they need different kinds of data.

I'm reviewing the code changes again. And found that seperate configuration for each exporter it's already supported. Sorry for the misunderstanding.

@lzf575 lzf575 assigned sharang and jiekun and unassigned jiekun Sep 8, 2023
@sharang sharang enabled auto-merge (rebase) September 8, 2023 01:27
@sharang sharang merged commit 29ebebd into main Sep 8, 2023
7 checks passed
@sharang sharang deleted the feature-support-multi-exporters branch September 8, 2023 01:30
@lzf575 lzf575 changed the title [ingester] support multiple OTLP exporters [ingester] support multiple OTLP exporters #4230 Dec 25, 2023
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.

3 participants