dukbong opened a new pull request, #1772: URL: https://github.com/apache/hertzbeat/pull/1772
## What's changed? <!-- Describe Your PR Here --> - Replaced the deprecated methods in RedisCommonCollectImpl with the builder pattern for RedisURI creation. This enhancement improves the maintainability and extensibility of the code. - Eliminated unnecessary string object allocations and enhanced readability using forEach and lambda expressions. This leads to code simplification and improved comprehension. - Improved readability by using StringUtils.isAnyEmpty() method. This helps clarify the intent of the code and prevents potential bugs. These three changes enhance the quality of the code, allowing for better maintainability and extensibility. ## Checklist - [ ] I have read the [Contributing Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide) - [ ] I have written the necessary doc or comment. - [ ] I have added the necessary unit tests and all cases have passed. ## Add or update API - [ ] I have added the necessary [e2e tests](https://github.com/apache/hertzbeat/tree/master/e2e) and all cases have passed. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
