89799969 opened a new pull request, #4049:
URL: https://github.com/apache/rocketmq-dashboard/pull/4049

   ## What changed
   
   - let Spring's default multi-argument key generator identify paged 
data-source queries
   - add a Spring caching regression test that distinguishes an absent search 
filter from the literal text `null`
   
   ## Why
   
   The previous delimiter-based key rendered Java `null` as the text `null`. An 
unfiltered request and a request searching for that literal value therefore 
shared one cache entry, so the latter could receive stale unfiltered results 
without reaching the repository. The default `SimpleKey` preserves argument 
boundaries and null values.
   
   Fixes #4048
   
   ## Verification
   
   - Before the production change: `SettingsServiceCachingTest` failed because 
the second query returned the first cached page.
   - `mvn '-Dtest=SettingsServiceCachingTest,SettingsServiceTest' test`
     - 44 tests, 0 failures, 0 errors, 0 skipped
     - Checkstyle: 0 violations
   
   ## Scope
   
   2 files changed; no API or persistence schema changes.


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