Croway commented on code in PR #9210:
URL: https://github.com/apache/camel/pull/9210#discussion_r1087689229
##########
components/camel-atmosphere-websocket/src/main/java/org/apache/camel/component/atmosphere/websocket/WebsocketEndpoint.java:
##########
@@ -86,15 +86,15 @@ public void setSendToAll(boolean sendToAll) {
this.sendToAll = sendToAll;
}
- public boolean isUseStreaming() {
- return useStreaming;
+ public boolean isStreaming() {
+ return streaming;
}
/**
* To enable streaming to send data as multiple text fragments.
*/
- public void setUseStreaming(boolean useStreaming) {
- this.useStreaming = useStreaming;
+ public void setStreaming(boolean streaming) {
Review Comment:
Hi @essobedo thanks a lot for your review, you were right, useStreaming was
not the issue, after some time debugging it, I noticed that Atmosphere builds
an in-memory map for each consumer but, without this information
https://github.com/apache/camel/pull/9210/files#diff-c20b148a113e2656ed364867af72660ee3204358f5ab036e1f4ac3c810f5c88fR62,
the path, the logic in Atmosphere is not enough, and messages are redirected
to the first entry of the map.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
