Tejaskriya commented on code in PR #8260:
URL: https://github.com/apache/ozone/pull/8260#discussion_r2137128423


##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/OMConfigKeys.java:
##########
@@ -627,4 +627,22 @@ private OMConfigKeys() {
   public static final String OZONE_OM_EDEKCACHELOADER_MAX_RETRIES_KEY =
       "ozone.om.edekcacheloader.max-retries";
   public static final int OZONE_OM_EDEKCACHELOADER_MAX_RETRIES_DEFAULT = 10;
+
+  /**
+   * Configuration properties for Compaction Service.
+   */
+  public static final String OZONE_OM_COMPACTION_SERVICE_ENABLED = 
"ozone.compaction.service.enabled";

Review Comment:
   Although it is consistent with what the config is named in 
ozone-default.xml, I will correct it in a followup jira. Thanks for catching 
this!
   
   @jojochuang I am not sure if someone using this version of the patch would 
face any issues. Do you happen to know if not following the config naming 
convention would cause any breakage?



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

Reply via email to