Skip to content

Support for HTTP Async client transport for Jersey on Scala (legacy master)#418

Merged
fehguy merged 1 commit intoswagger-api:masterfrom
lucamilanesio:scala-async-http-support
Feb 9, 2015
Merged

Support for HTTP Async client transport for Jersey on Scala (legacy master)#418
fehguy merged 1 commit intoswagger-api:masterfrom
lucamilanesio:scala-async-http-support

Conversation

@lucamilanesio
Copy link
Contributor

When generating Scala client, the ApiInvoker is a class with
overridable options, avoiding the need of public vars to be
manually modified.

Additionally it is possible to plug the AsyncHttp protocol
for Jersey in order to achieve Netty support and maximum scalability.

@lucamilanesio lucamilanesio force-pushed the scala-async-http-support branch from 88f1f46 to 0976a1c Compare February 9, 2015 19:29
@fehguy
Copy link
Contributor

fehguy commented Feb 9, 2015

this is great--we're mostly working in the develop_2.0 branch and that would be the appropriate place to merge this. Can you send the PR there?

When generating Scala client, the ApiInvoker is a class with
overridable options, avoiding the need of public vars to be
manually modified.

Additionally it is possible to plug the AsyncHttp protocol
for Jersey in order to achieve Netty support and maximum scalability.

NOTE: Support for Scala 2.10.0-3 removed as largely obsolete
@lucamilanesio lucamilanesio force-pushed the scala-async-http-support branch from 0976a1c to 2386e39 Compare February 9, 2015 19:36
@lucamilanesio
Copy link
Contributor Author

@fehguy thanks for the feedback: I would keep this PR anyway for a fix on the release we are currently using and additionally I'll open a new one on develop_2.0

@lucamilanesio lucamilanesio changed the title Support for HTTP Async client transport for Jersey on Scala Support for HTTP Async client transport for Jersey on Scala (legacy master) Feb 9, 2015
fehguy added a commit that referenced this pull request Feb 9, 2015
Support for HTTP Async client transport for Jersey on Scala (legacy master)
@fehguy fehguy merged commit ece072a into swagger-api:master Feb 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants