Skip to content

WildFly : REST client doesn't work, XMLHttpRequest need to be reopen before each request #97

Merged
arun-gupta merged 2 commits intojavaee-samples:masterfrom
mgreau:master
Nov 10, 2013
Merged

WildFly : REST client doesn't work, XMLHttpRequest need to be reopen before each request #97
arun-gupta merged 2 commits intojavaee-samples:masterfrom
mgreau:master

Conversation

@mgreau
Copy link
Copy Markdown
Contributor

@mgreau mgreau commented Nov 10, 2013

I try to test this app with WildFly.
There is no problem for WebSocket test but REST client doesn't work (just one call works).

I'm using Google Chrome.
It seems that XMLHttpRequest need to be reopen before each POST request.

With this code, benchmark works on my laptop.

arun-gupta added a commit that referenced this pull request Nov 10, 2013
WildFly : REST client doesn't work, XMLHttpRequest need to be reopen before each request
@arun-gupta arun-gupta merged commit 00f6bcf into javaee-samples:master Nov 10, 2013
@arun-gupta
Copy link
Copy Markdown
Contributor

Can you share the benchmark numbers ?

@mgreau
Copy link
Copy Markdown
Contributor Author

mgreau commented Nov 10, 2013

You can find it on my blog post, next to yours :)
http://mgreau.com/posts/2013/11/11/javaee7-websocket-angularjs-wildfly.html#benchmark-websocket-vs-rest

Maxime Gréau.

2013/11/10 Arun Gupta notifications@github.com

Can you share the benchmark numbers ?


Reply to this email directly or view it on GitHubhttps://github.com//pull/97#issuecomment-28159712
.

@arun-gupta
Copy link
Copy Markdown
Contributor

sweet, numbers look awefully familiar as on my machine :-)

Cool, thanks for publishing them.

Do the benchmark source code looks good to you ? Any suggestions for
improvement ?

On Sun, Nov 10, 2013 at 10:33 PM, Maxime Gréau notifications@github.comwrote:

You can find it on my blog post, next to yours :)

http://mgreau.com/posts/2013/11/11/javaee7-websocket-angularjs-wildfly.html#benchmark-websocket-vs-rest

Maxime Gréau.

2013/11/10 Arun Gupta notifications@github.com

Can you share the benchmark numbers ?


Reply to this email directly or view it on GitHub<
https://github.com/arun-gupta/javaee7-samples/pull/97#issuecomment-28159712>

.


Reply to this email directly or view it on GitHubhttps://github.com//pull/97#issuecomment-28159784
.

http://blog.arungupta.me
http://twitter.com/arungupta

@mgreau
Copy link
Copy Markdown
Contributor Author

mgreau commented Nov 11, 2013

Yes :), it will be interesting to compare the amount of data exchanged (except payload data) in both case.
I mean headers HTTP (request+response) compared to WebSocket data.

I started to look at this http://www.w3.org/TR/XMLHttpRequest/#the-getallresponseheaders%28%29-method.
I will try to do it when I'll have a moment :)

@arun-gupta
Copy link
Copy Markdown
Contributor

Yeah, do you want to send a pull request for that test case ?

BTW, did you get some increased traffic to your blog becauseof multiple RTs
;)

On Mon, Nov 11, 2013 at 10:35 PM, Maxime Gréau notifications@github.comwrote:

Yes :), it will be interesting to compare the amount of data exchanged
(except payload data) in both case.
I mean headers HTTP (request+response) compared to WebSocket data.

I started to look at this
http://www.w3.org/TR/XMLHttpRequest/#the-getallresponseheaders%28%29-method
.
I will try to do it when I'll have a moment :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/97#issuecomment-28241588
.

http://blog.arungupta.me
http://twitter.com/arungupta

@mgreau
Copy link
Copy Markdown
Contributor Author

mgreau commented Nov 11, 2013

Yes, that's amazing, thanks for your tweets :)
visits

For the pull request, I didn't start to code.
I wonder if the processing can only be done on the client side.

But currently I try to improve my "when websocket met asciidoctor" application :)

@arun-gupta
Copy link
Copy Markdown
Contributor

Awesome, really happy for you!

I think a comparison of amount of headers exchanged for HTTP and WebSocket
would be really useful. Created
#98, send a pull
request when ever ready.

On Mon, Nov 11, 2013 at 11:08 PM, Maxime Gréau notifications@github.comwrote:

Yes, that's amazing, thanks for your tweets :)
[image: visits]https://f.cloud.github.com/assets/501525/1517025/db39cbaa-4b1c-11e3-8236-8dc580bc1c03.png

For the pull request, I don't start to code.
I wonder if the treatment can only be done on the client side.

But currently I try to improve my "when websocket met asciidoctor"
application :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/97#issuecomment-28244435
.

http://blog.arungupta.me
http://twitter.com/arungupta

@mgreau
Copy link
Copy Markdown
Contributor Author

mgreau commented Nov 11, 2013

thanks.

Ok I will do it.

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