Copilot commented on code in PR #9495:
URL: https://github.com/apache/ozone/pull/9495#discussion_r2632340630
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java:
##########
@@ -376,6 +376,11 @@ public void start(OzoneConfiguration configuration) {
*/
public void startSnapshotSstFilteringService(OzoneConfiguration conf) {
if (isSstFilteringSvcEnabled()) {
+ if (isDefragSvcEnabled()) {
+ LOG.info("SstFilteringService is disabled (despite config intends to
enable it) " +
Review Comment:
The log message contains grammatical issues. The phrase "despite config
intends to enable it" should be "despite configuration intending to enable it"
for proper grammar. Consider rephrasing to: "SstFilteringService is disabled
(despite configuration intending to enable it) because SnapshotDefragService is
enabled. Defrag effectively performs filtering already."
```suggestion
LOG.info("SstFilteringService is disabled (despite configuration
intending to enable it) " +
```
--
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]
