Vertx server adapter

[vietj]

Integration with Vert.x 3.3

[jimma]

Hi Julien,
Is callEmpty(client) in JaxrsAsyncTest for 3 times intended ?
For the new added RESTEASY1323Test , RESTEASY1325Test, if these issues are already verified by tests in resteasy-netty4 module, do we need to check in vertx module again ?
Thanks

[vietj]

@jimma to be honest I just copied the Netty tests as Vert.x and Netty are very similar, we can remove these tests if you think it's better - I have no strong opinion on this.

[vietj]

@jimma about callEmpty called 3 times, again I copied the Netty 4 tests, that do the same

[jimma]

I'll merge and fix these tests later.

[vietj]

@jimma thanks!

[vietj]

@jimma any idea when this will be in a Resteasy release ?

[asoldano]

Likely in a beta release by the end of this week

[vietj]

great, don't hesitate to ping us when it's done and perhaps you can announce it to the Vert.x google group as well ?

[asoldano]

I've committed a small fix to change the scope of a maven dependency, as the merged commits were breaking the build.
Besides that, I've created https://issues.jboss.org/browse/RESTEASY-1453 , can you please fill in the description / provide some details so that the addition is tracked in the release notes?
It would also be cool if you could provide a bit of documentation, otherwise we'll manage to write something ;-)

[vietj]

Hi,

I added a section in the Resteasy docbook, what else do you expect ?

On Jul 26, 2016, at 4:45 PM, Alessio Soldano notifications@github.com wrote:

I've committed a small fix to change the scope of a maven dependency, as the merged commits were breaking the build.
Besides that, I've created https://issues.jboss.org/browse/RESTEASY-1453 https://issues.jboss.org/browse/RESTEASY-1453 , can you please fill in the description / provide some details so that the addition is tracked in the release notes?
It would also be cool if you could provide a bit of documentation, otherwise we'll manage to write something ;-)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #868 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AANxiiMSWB3xjj6Zpg2oY7C3Ii2WzDKrks5qZh2ZgaJpZM4JMyHG.

[vietj]

I have added a comment as I could not find a way to edit a description in the JIRA task.

On Jul 26, 2016, at 4:45 PM, Alessio Soldano notifications@github.com wrote:

I've committed a small fix to change the scope of a maven dependency, as the merged commits were breaking the build.
Besides that, I've created https://issues.jboss.org/browse/RESTEASY-1453 https://issues.jboss.org/browse/RESTEASY-1453 , can you please fill in the description / provide some details so that the addition is tracked in the release notes?
It would also be cool if you could provide a bit of documentation, otherwise we'll manage to write something ;-)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #868 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AANxiiMSWB3xjj6Zpg2oY7C3Ii2WzDKrks5qZh2ZgaJpZM4JMyHG.

[asoldano]

@vietj thanks; actually I missed the changed to the docbook, so we're fine :-)
Last thing, the VertxContainer is in package org.jboss.resteasy.test... I assume that's a mistake and I'd move that to package org.jboss.resteasy.plugins.server.vertx, ok?

[jimma]

@vietj , I removed some tests from your change and clean up some imports. Can you please have a look at this PR #883 ? If it's OK, I am going to merge it. Thanks.

[vietj]

@jimma n/p with me, how does 880 relates to vertx ?

[vietj]

@asoldano ok for moving this to a more appropriate place (I copied what netty did)

[jimma]

Should be PR #883. Sorry.

@jimma n/p with me, how does 880 relates to vertx ?