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.

Sign up for GitHub

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

Jump to bottom

Do not waitForResponse in OtlpHttpClient::Export ? #1176

Closed
owent opened this issue Jan 17, 2022 · 5 comments
Closed

Do not waitForResponse in OtlpHttpClient::Export ? #1176

owent opened this issue Jan 17, 2022 · 5 comments
Assignees
@owent
Labels
do-not-stale

Comments

@owent
Copy link
Member

owent commented Jan 17, 2022 •
edited
Loading

Is your feature request related to a problem?

Mentioned in #1175 .

Describe the solution you'd like

Do not waitForResponse in OtlpHttpClient::Export and destroy session only after ResponseHandler got some events.
We also need control parallel thread spawned by HttpOperation::SendAsync here.

This can also assign to me.
@owent owent mentioned this issue Jan 17, 2022
Closed
@owent
Copy link
Member Author

owent commented Jan 24, 2022

I'm a little busy recently and I may finish this several days later.

@owent owent mentioned this issue Feb 13, 2022
Closed
3 tasks
@owent owent mentioned this issue Mar 4, 2022
Closed
@github-actions
Copy link

This issue was marked as stale due to lack of activity. It will be closed in 7 days if no furthur activity occurs.

@github-actions github-actions bot added the Stale label Mar 26, 2022
@lalitb lalitb removed the Stale label Mar 26, 2022
@owent owent mentioned this issue May 21, 2022
Merged
3 tasks
@github-actions
Copy link

This issue was marked as stale due to lack of activity. It will be closed in 7 days if no furthur activity occurs.

@github-actions github-actions bot added the Stale label May 25, 2022
@github-actions
Copy link

github-actions bot commented Jun 1, 2022

Closed as inactive. Feel free to reopen if this is still an issue.

@github-actions github-actions bot closed this as completed Jun 1, 2022
@lalitb lalitb reopened this Jun 1, 2022
@github-actions
Copy link

github-actions bot commented Jun 9, 2022

Closed as inactive. Feel free to reopen if this is still an issue.

@github-actions github-actions bot closed this as completed Jun 9, 2022
@lalitb lalitb reopened this Jun 9, 2022
@lalitb lalitb added do-not-stale and removed Stale labels Jun 9, 2022
@owent owent closed this as completed Jul 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@owent owent

Labels
do-not-stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Concurrency HTTP sessions for OtlpHttpClient owent/opentelemetry-cpp
2 participants
@lalitb @owent