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/datasetexporter]: Mark as alpha #22154

Merged

Conversation

martin-majlis-s1
Copy link
Contributor

@martin-majlis-s1 martin-majlis-s1 commented May 22, 2023

Description: Change status to alpha

Based on the instructions - the last step should be:

Previous PRs:

Link to tracking Issue: #20660

Testing:

  • we are running this version for few weeks on our dev environments
  • we are running open telemetry demo with our custom build image
  • based on my initial benchmark it can export on my notebook around 2MB of logs per second

Documentation:

There are no updates in the documentation - it has still the same configuration.

@martin-majlis-s1
Copy link
Contributor Author

@mx-psi : Since you have been reviewing, approving and merging previous PRs, could you, please, take a look on this PR?

Copy link
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, will wait for @atoulme to have a look as well on this one

@dmitryax dmitryax merged commit 721e6b0 into open-telemetry:main May 22, 2023
@github-actions github-actions bot added this to the next release milestone May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants