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

   ## What changes
   
   The non-WebSocket `AbstractDataChangedListener` callbacks previously 
selected the namespace from the first changed record, then refreshed that 
namespace cache and passed the entire batch to the channel-specific callback. 
For mixed-namespace batches, this could publish records under the wrong 
namespace and leave other namespace caches stale.
   
   Group app auth, metadata, plugin, rule, selector, proxy selector, and 
discovery-upstream changes by normalized namespace. Refresh each namespace 
cache and invoke its callback with only that namespace’s records. Preserve the 
default namespace fallback and skip null records with a warning.
   
   This is the companion follow-up requested in the review of #7330 for the 
common listener used by non-WebSocket sync channels. It complements the 
WebSocket sender fix and addresses the same mixed-namespace behavior described 
in #7314.
   
   ## Validation
   
   - `./mvnw -pl shenyu-admin -am -Dtest=AbstractDataChangedListenerTest 
-Dsurefire.failIfNoSpecifiedTests=false -Djacoco.skip=true 
-Dmaven.javadoc.skip=true test` (38-module reactor; 10 tests passed; Checkstyle 
clean)
   - `git diff --check`


-- 
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