Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Resin doesn't support removing the encoding part of the HTTP content header for the reply #6

Closed
zloster opened this issue Mar 22, 2017 · 3 comments
Closed

Resin doesn't support removing the encoding part of the HTTP content header for the reply #6

zloster opened this issue Mar 22, 2017 · 3 comments
Labels
wontfix
Milestone
Better Times

Comments

@zloster
Copy link
Owner

zloster commented Mar 22, 2017

This affects the plaintext results. I can't be sure to what extent. But 330 000+ (Round 13, cloud environment, plaintext) replies per second with each reply has + 14 bytes it will add up.
@zloster zloster modified the milestone: BetterTimes Apr 1, 2017
zloster pushed a commit that referenced this issue Oct 2, 2018
@michaelhixson @NateBrady23
Fix an uncaught exception during dart startup (TechEmpower#3640)
04dd98d 
Here's an example of the error output from before this change:

Unhandled exception:
Bad state: Stream was already listened to
#0      _RawServerSocket.listen (dart:io-patch/socket_patch.dart:1106)
#1      new _ForwardingStreamSubscription (dart:async/stream_pipe.dart:123)
#2      _ForwardingStream._createSubscription (dart:async/stream_pipe.dart:91)
#3      _ForwardingStream.listen (dart:async/stream_pipe.dart:86)
#4      _ServerSocket.listen (dart:io-patch/socket_patch.dart:1351)
#5      _HttpServer.listen (dart:io/http_impl.dart:2278)
#6      _startServer.<anonymous closure> (file:///server.dart:88:12)
#7      _RootZone.runUnary (dart:async/zone.dart:1371)
#8      _FutureListener.handleValue (dart:async/future_impl.dart:129)
#9      _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:636)
#10     _Future._propagateToListeners (dart:async/future_impl.dart:665)
#11     _Future._completeWithValue (dart:async/future_impl.dart:478)
#12     Future.wait.<anonymous closure> (dart:async/future.dart:362)
#13     _RootZone.runUnary (dart:async/zone.dart:1371)
#14     _FutureListener.handleValue (dart:async/future_impl.dart:129)
#15     _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:636)
#16     _Future._propagateToListeners (dart:async/future_impl.dart:665)
#17     _Future._completeWithValue (dart:async/future_impl.dart:478)
#18     Future.wait.<anonymous closure> (dart:async/future.dart:362)
#19     _RootZone.runUnary (dart:async/zone.dart:1371)
#20     _FutureListener.handleValue (dart:async/future_impl.dart:129)
#21     _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:636)
#22     _Future._propagateToListeners (dart:async/future_impl.dart:665)
#23     _Future._completeWithValue (dart:async/future_impl.dart:478)
#24     _Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:510)
#25     _microtaskLoop (dart:async/schedule_microtask.dart:41)
#26     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)
#27     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:99)
#28     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:152)
@zloster
Copy link
Owner Author

zloster commented Oct 12, 2018

The numbers are quite higher in 2018 - currently 2 500 000 rps. Revenj.jvm achieves 3 100 000 rps. Gemini however is at 1 500 000 rps at max. Quite a big difference.

@zloster
Copy link
Owner Author

zloster commented Oct 12, 2018

There is no big difference:
https://github.com/TechEmpower/FrameworkBenchmarks/blob/a8bc95124b7bd39b93ef65de5c60c67bc683ff62/f!rameworks/Java/revenj-jvm/src/main/java/hello/PlaintextServlet.java#L10-L16

https://github.com/TechEmpower/FrameworkBenchmarks/blob/a8bc95124b7bd39b93ef65de5c60c67bc683ff62/frameworks/Java/servlet/src/main/java/hello/PlaintextServlet.java#L15-L33
Without using a constant is better?!

@zloster
Copy link
Owner Author

zloster commented May 17, 2019

The verification code doesn't flag this as problem for some time now. So closing.

@zloster zloster closed this as completed May 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
wontfix
Projects
None yet
Milestone
Better Times
Development

No branches or pull requests
1 participant
@zloster