flying-musk opened a new pull request, #400:
URL: https://github.com/apache/rocketmq-dashboard/pull/400

   ## What is the purpose of the change
   
   This PR makes resource loading in `AbstractFileStore` use an explicit class 
reference instead of `getClass()`.
   
   The current code uses:
   
   `getClass().getResourceAsStream("/" + fileName)`
   
   Since `AbstractFileStore` is designed to be extended, this makes the lookup 
depend on the runtime subclass, which can introduce inheritance-related 
ambiguity and make the behavior less explicit for future maintenance.
   
   Using `AbstractFileStore.class.getResourceAsStream(...)` makes the resource 
lookup context explicit and stable.
   
   ## Brief changelog
   
   - Replace `getClass().getResourceAsStream("/" + fileName)` with 
`AbstractFileStore.class.getResourceAsStream("/" + fileName)` in 
`AbstractFileStore`
   
   ## Verifying this change
   
   - Verified that the project still compiles after the change
   - Verified by running `mvn test`


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