Skip to content

Add Kotlin Coroutine Support #106

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

Closed
cgm-aw opened this issue Mar 12, 2024 · 1 comment
Closed

Add Kotlin Coroutine Support #106

cgm-aw opened this issue Mar 12, 2024 · 1 comment
Labels
question Further information is requested

Comments

@cgm-aw
Copy link

cgm-aw commented Mar 12, 2024

Hi there,

I'm using this project for contextual logging and it works fine in most cases, however my stack is based on Kotlin and Coroutines.
Most of the time the context data is correct, but there are cases where the data gets mixed up with other context data.
I did not yet succeed in building a reproducer since it only seems to happen in complex call chains under some load.
However I assume a fix would be by using Kotlins CoroutineContext instead of Vertx' ContextInternal as described for Open Tracing.

Are there plans to support coroutines?

Best regards

@tsegismont tsegismont added the question Further information is requested label Mar 13, 2024
@tsegismont
Copy link
Contributor

There are no plans to implement a contextual data storage with the specific Kotlin coroutine context.

If you manage to put together a small reproducer, I'd be glad to look into it.

@tsegismont tsegismont closed this as not planned Won't fix, can't repro, duplicate, stale Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants