|<< Back to previous view|
[TYRUS-43] MessageHandler.CharacterStream onMessage method called for every received partial string Created: 23/Oct/12 Updated: 08/Mar/13 Resolved: 08/Mar/13
|Fix Version/s:||1.0-b13, 1.0|
|Reporter:||Martin Matula||Assignee:||Pavel Bucek|
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
|Participants:||dannycoward, Martin Matula and Pavel Bucek|
see BlockingStreamingTextTest - it is currently ignored, as it breaks due to this bug.
The client sends partial strings, and on the server side, it seems that for every partial string received, a new call to onMessage() method is made. I believe this is wrong (flawed) as the CharacterStream.onMessage() method has no means to determine when the last part of the string came - so IMO should receive the full string (i.e. keep receiving string parts until the last is received) rather than invoking new handler for every part.
Once this bug is fixed, the @Ignore annotation should be removed from the test so that it runs as part of the build.
|Comment by dannycoward [ 26/Oct/12 11:35 PM ]|
The endpointwrapper was creating a new blocking adapter for each partial message. Now this is fixed.
The same issue held for blocking binary data, which the fix for this also fixes.
I am holding off closing this issue until I've seen it work in hudson. Its working great in local and workspace tests.
|Comment by Pavel Bucek [ 05/Feb/13 10:10 AM ]|
still not resolved in trunk.