fapifta commented on code in PR #6860:
URL: https://github.com/apache/ozone/pull/6860#discussion_r1675995044
##########
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/conf/OzoneConfiguration.java:
##########
@@ -407,4 +420,76 @@ public int getInt(String name, String fallbackName, int
defaultValue,
}
return Integer.parseInt(value);
}
+
+ private Properties delegatingProps;
+
+ @Override
+ public synchronized void reloadConfiguration() {
+ super.reloadConfiguration();
+ delegatingProps = null;
+ }
+
+ @Override
+ protected final synchronized Properties getProps() {
+ if (delegatingProps == null) {
+ delegatingProps = new DelegatingProperties(super.getProps(),
complianceMode, cryptoProperties);
+ }
+ return delegatingProps;
+ }
+
+ /**
+ * Get a property value without the compliance check. It's needed to get the
compliance
+ * mode and the whitelist parameter values in the checkCompliance method.
+ *
+ * @param key property name
+ * @param defaultValue default value
+ * @return property value, without compliance check
+ */
+ private String getPropertyUnsafe(String key, String defaultValue) {
+ return super.getProps().getProperty(key, defaultValue);
+ }
+
+ private Properties getCryptoProperties() {
+ try {
+ return
super.getAllPropertiesByTag(ConfigTag.CRYPTO_COMPLIANCE.toString());
+ } catch (NoSuchMethodError e) {
Review Comment:
thank you, it looks good.
--
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]
