kalayciburak opened a new pull request, #7330:
URL: https://github.com/apache/shenyu/pull/7330

   ## Summary
   - `WebsocketDataChangedListener` took the namespace from the first record 
and sent the whole batch to that namespace only.
   - Mixed batches are now partitioned by effective namespace before the 
payload is built. Each namespace receives only its own records, independent of 
input order.
   - A null namespace still falls back to the default namespace. Null records 
are skipped. A single-namespace batch is still one message.
   
   `HttpLongPollingDataChangedListener` keys off the client request, not the 
batch. `AbstractDataChangedListener` and `AbstractNodeDataChangedListener` 
still use the first record's namespace for cache refresh and node writes; that 
is a separate path from this websocket send.
   
   Fixes #7314
   
   ## Test plan
   - [x] `JAVA_HOME=17 ./mvnw -pl shenyu-admin -am test 
-Dtest=WebsocketDataChangedListenerNamespaceTest,WebsocketDataChangedListenerTest
 -Dsurefire.failIfNoSpecifiedTests=false -Djacoco.skip=true` — 30 tests, 0 
failures. Checkstyle validate on that reactor run reported 0 violations.
   - [ ] Docker e2e was not run.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to