sadadw1 opened a new pull request, #678:
URL: https://github.com/apache/ozhera/pull/678
### Ⅰ. Describe what this PR did
Corrects a misspelling in a log message of `HeraBaseInfoServiceImpl`. The
string `sucess` is changed to `success` in the "del HeraAppRole AppById" info
log.
```diff
- log.info("del HeraAppRole AppById sucess!dataId:{}", id);
+ log.info("del HeraAppRole AppById success!dataId:{}", id);
```
This is a log-text-only change and does not affect any business logic.
### Ⅱ. Does this pull request fix one issue?
fixes #677
### Ⅲ. Why don't you add test cases (unit test/integration test)?
Text-only typo fix in a log string; no behavior change to cover with a test.
### Ⅳ. Describe how to verify it
Grep the source to confirm no `sucess` remains in this file, and that the
surrounding delete flow is untouched.
### Ⅴ. Special notes for reviews
Note: a different occurrence of `sucess` in `AppMonitorController` was
already fixed by #672. This PR addresses the remaining occurrence in
`HeraBaseInfoServiceImpl`.
--
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]
