kgeisz opened a new pull request, #8115:
URL: https://github.com/apache/hbase/pull/8115
This pull request addresses several comments in the review for PR #8044.
This PR makes the following changes:
1. Re-introduced a log message indicating coprocessors are being updated due
to a configuration change. This log message has been moved to
* https://github.com/apache/hbase/pull/8044#discussion_r3054311082
2. Read-Only mode is now tracked by checking whether the ReadOnlyController
coprocessors are present, rather than tracking with an instance variable in
HMaster, HRegionServer, and HRegion. If the coprocessors are present, then
read-only mode is assumed to be active.
* https://github.com/apache/hbase/pull/8044#discussion_r3054336616
3. Removed dynamic configuration for coprocessors, which was introduced in
PR #6931.
* https://github.com/apache/hbase/pull/8044#discussion_r3054354155
4. The `onConfigurationChange()` method for HMaster, HRegionServer, and
HRegion now use the lambda function's `conf` variable in
`maybeUpdateCoprocessors()`.
* https://github.com/apache/hbase/pull/8044#discussion_r3054364562
5. Removes extra portion of comment for `HBASE_META_TABLE_SUFFIX`.
* https://github.com/apache/hbase/pull/8044#discussion_r3082492374
--
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]
