sharmaar12 commented on code in PR #7743:
URL: https://github.com/apache/hbase/pull/7743#discussion_r2834746860
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java:
##########
@@ -885,6 +888,17 @@ public HRegion(final HRegionFileSystem fs, final WAL wal,
final Configuration co
: this.htableDescriptor.getDurability();
decorateRegionConfiguration(conf);
+
+ if (
+ conf.getBoolean(HConstants.HBASE_GLOBAL_READONLY_ENABLED_KEY,
+ HConstants.HBASE_GLOBAL_READONLY_ENABLED_DEFAULT)
+ ) {
+ addReadOnlyCoprocessors(this.baseConf);
+ addReadOnlyCoprocessors(this.conf);
+ } else {
+ removeReadOnlyCoprocessors(this.baseConf);
Review Comment:
We need to add this to introduce the REGION_COPROCESSOR_CONF_KEY property in
the conf object, which is instance of CompoundConfiguration. This is needed
because if the property is not present as a local copy in CompoundConfiguration
then updating it using setStrings does not work so though Coprocessors are not
needed still the property must be set to empty collection.
--
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]
