-
Notifications
You must be signed in to change notification settings - Fork 59
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
Run tests on Netty for WebFlux #112
Comments
Is this causing problems for you for some reason? |
yes, it looks like TracingSubscriber.onComplete() is never called, therefore span is not finished: |
So I might be missing something here... Is this tied to another issue perhaps? |
If I use in my app tomcat then I see that |
@csabakos can you shed some light on this perhaps since you were the one that added those tests? Thanks |
@geoand WebFlux should work equally over any runtime (Tomcat, Jetty, Netty, etc.), so there was no reason to switch away from the runtime that was already used in other tests. If the observed behavior is different on Netty, then I think it would make sense to add Netty-based integration tests to make sure that all cases are covered. |
So you just added like that for convenience, thanks that's good to know. @malafeev Would you like to take a stab at it? If you are unavailable, I can take a look but it will probably be a while... |
@geoand it would be great if you take a look even in a while. Currently I don't have enough time. |
Same here :). I'll take a look as soon as I can |
I want to upgrade to Opentracing 0.33 first. Then I'll look into this |
WebFlux doesn't require jetty or tomcat.
It uses netty.
Is it possible to get rid of jetty from WebFlux integration tests?
The text was updated successfully, but these errors were encountered: