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

[exporter/kafka] do not expose method #27583

Merged
merged 2 commits into from Oct 26, 2023

Conversation

sakulali
Copy link
Contributor

Description:
Do not export function WithTracesMarshalers, WithMetricsMarshalers, WithLogsMarshalers and pass checkapi

Link to tracking Issue:
#26304

Testing:
go run cmd/checkapi/main.go .
make chlog-validate
go test for kafkaexporter

Documentation:

@github-actions
Copy link
Contributor

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

@github-actions github-actions bot added the Stale label Oct 26, 2023
@songy23 songy23 added ready to merge Code review completed; ready to merge by maintainers and removed Stale labels Oct 26, 2023
@codeboten codeboten merged commit a72128b into open-telemetry:main Oct 26, 2023
83 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 26, 2023
sigilioso pushed a commit to carlossscastro/opentelemetry-collector-contrib that referenced this pull request Oct 27, 2023
Do not export function WithTracesMarshalers, WithMetricsMarshalers,
WithLogsMarshalers and pass checkapi

open-telemetry#26304

**Testing:** 
go run cmd/checkapi/main.go .
make chlog-validate
go test for kafkaexporter


Co-authored-by: Alex Boten <aboten@lightstep.com>
@sakulali sakulali deleted the fix-kafkaexporter-api branch November 2, 2023 13:57
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2023
Do not export function WithTracesMarshalers, WithMetricsMarshalers,
WithLogsMarshalers and pass checkapi

open-telemetry#26304

**Testing:** 
go run cmd/checkapi/main.go .
make chlog-validate
go test for kafkaexporter


Co-authored-by: Alex Boten <aboten@lightstep.com>
RoryCrispin pushed a commit to ClickHouse/opentelemetry-collector-contrib that referenced this pull request Nov 24, 2023
Do not export function WithTracesMarshalers, WithMetricsMarshalers,
WithLogsMarshalers and pass checkapi

open-telemetry#26304

**Testing:** 
go run cmd/checkapi/main.go .
make chlog-validate
go test for kafkaexporter


Co-authored-by: Alex Boten <aboten@lightstep.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/kafka ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants