[
https://issues.apache.org/jira/browse/HBASE-28334?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bryan Beaudreault updated HBASE-28334:
--------------------------------------
Fix Version/s: 2.6.0
> Add comment around erasure coding policy in DEFAULT_VALUE map
> --------------------------------------------------------------
>
> Key: HBASE-28334
> URL: https://issues.apache.org/jira/browse/HBASE-28334
> Project: HBase
> Issue Type: Improvement
> Reporter: Bryan Beaudreault
> Assignee: Nirdosh Kumar Yadav
> Priority: Major
> Labels: pull-request-available
> Fix For: 2.6.0, 3.0.0-beta-2
>
>
> New description: Upon investigation, we realized that it's necessary to put
> ERASURE_CODING_POLICY in the DEFAULT_VALUES map so that when the
> TableDescriptor is printed as a string, that setting is not listed as
> METADATA. We added a comment to note that for future reference.
> Old description: With ERASURE_CODING_POLICY, the default value is null (no
> policy). I added a record of that in DEFAULT_VALUES, because other settings
> seemed to do that.
> A null value is never stored on a HTD because our code handles removing from
> map when setting null. So we'd never have an opportunity to match against the
> DEFAULT_VALUE. If someone tried setting a string value "null", that would
> fail validation because it's not a valid policy. So there's no reason to
> record this default value. It doesn't cause a problem, but is confusing to
> anyone reading the code. Remove it.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
