kgeisz commented on code in PR #8115:
URL: https://github.com/apache/hbase/pull/8115#discussion_r3139121308
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/util/CoprocessorConfigurationUtil.java:
##########
@@ -182,13 +181,14 @@ private static List<String>
getReadOnlyCoprocessors(String configurationKey) {
/**
* This method adds or removes relevant ReadOnlyController coprocessors to
the provided
- * configuration based on whether read-only mode is enabled.
+ * configuration based on whether read-only mode is enabled in the provided
Configuration.
* @param conf The up-to-date configuration used to determine
how to handle
* coprocessors
* @param coprocessorConfKey The configuration key name
*/
public static void syncReadOnlyConfigurations(Configuration conf, String
coprocessorConfKey) {
- boolean isReadOnlyModeEnabled =
ConfigurationUtil.isReadOnlyModeEnabled(conf);
+ boolean isReadOnlyModeEnabled =
conf.getBoolean(HConstants.HBASE_GLOBAL_READONLY_ENABLED_KEY,
Review Comment:
Yes, definitely. Thanks!
--
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]
