[jira] [Updated] (HBASE-28624) Docs around configuring backups can lead to unexpectedly disabling other features

["Bryan Beaudreault (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HBASE-28624?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bryan Beaudreault updated HBASE-28624:
--------------------------------------
    Issue Type: Improvement  (was: Bug)

> Docs around configuring backups can lead to unexpectedly disabling other features
> ---------------------------------------------------------------------------------
>
>                 Key: HBASE-28624
>                 URL: https://issues.apache.org/jira/browse/HBASE-28624
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Bryan Beaudreault
>            Priority: Major
>
> In our documentation for enabling backups, we suggest that the user set the following:
> {code:java}
> <property>
>   <name>hbase.master.logcleaner.plugins</name>
>   <value>org.apache.hadoop.hbase.backup.master.BackupLogCleaner,...</value>
> </property>
> <property>
>   <name>hbase.master.hfilecleaner.plugins</name>
>   <value>org.apache.hadoop.hbase.backup.BackupHFileCleaner,...</value>
> </property> {code}
> A naive user will set these and not know what to do about the ",..." part. In doing so, they will unexpectedly be disabling all of the default cleaners we have. For example here are the defaults:
> {code:java}
> <property>
>   <name>hbase.master.logcleaner.plugins</name>
>   <value>org.apache.hadoop.hbase.master.cleaner.TimeToLiveLogCleaner,org.apache.hadoop.hbase.master.cleaner.TimeToLiveProcedureWALCleaner,org.apache.hadoop.hbase.master.cleaner.TimeToLiveMasterLocalStoreWALCleaner</value>
> </property>
> <property>
>   <name>hbase.master.hfilecleaner.plugins</name>
>   <value>org.apache.hadoop.hbase.master.cleaner.TimeToLiveHFileCleaner,org.apache.hadoop.hbase.master.cleaner.TimeToLiveMasterLocalStoreHFileCleaner</value>
> </property> {code}
> So basically disabling support for hbase.master.logcleaner.ttl and hbase.master.hfilecleaner.ttl.
> There exists a method BackupManager.decorateMasterConfiguration and BackupManager.decorateRegionServerConfiguration. They are currently javadoc'd as being for tests only, but I think we should call these in HMaster and HRegionServer. Then we can only require the user to set "hbase.backup.enable" and very much simplify our docs here.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)