[
https://issues.apache.org/jira/browse/HBASE-19433?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16281163#comment-16281163
]
Josh Elser commented on HBASE-19433:
------------------------------------
bq. The above change is for passing mockito type checking.
I assumed as much, but it seems silly to me to add a new method to return the
same object when we can work around it in the test with a cast. Am I missing
something?
> ChangeSplitPolicyAction modifies an immutable HTableDescriptor
> --------------------------------------------------------------
>
> Key: HBASE-19433
> URL: https://issues.apache.org/jira/browse/HBASE-19433
> Project: HBase
> Issue Type: Bug
> Components: integration tests
> Reporter: Josh Elser
> Assignee: Ted Yu
> Priority: Critical
> Fix For: 2.0.0-beta-1
>
> Attachments: 19433-unsupported.out, 19433.v1.txt, 19433.v2.txt,
> 19433.v3.txt, 19433.v4.txt
>
>
> {noformat}
> 2017-12-01 23:18:51,433 WARN [ChaosMonkeyThread] policies.Policy: Exception
> occurred during performing action: java.lang.UnsupportedOperationException:
> HTableDescriptor is read-only
> at
> org.apache.hadoop.hbase.client.ImmutableHTableDescriptor.getDelegateeForModification(ImmutableHTableDescriptor.java:59)
> at
> org.apache.hadoop.hbase.HTableDescriptor.setRegionSplitPolicyClassName(HTableDescriptor.java:333)
> at
> org.apache.hadoop.hbase.chaos.actions.ChangeSplitPolicyAction.perform(ChangeSplitPolicyAction.java:54)
> at
> org.apache.hadoop.hbase.chaos.policies.PeriodicRandomActionPolicy.runOneIteration(PeriodicRandomActionPolicy.java:59)
> at
> org.apache.hadoop.hbase.chaos.policies.PeriodicPolicy.run(PeriodicPolicy.java:41)
> at java.lang.Thread.run(Thread.java:745)
> {noformat}
> Found during some internal testing. Need to make sure this Action, in
> addition to the other, don't fall into the trap of modifying the
> TableDescriptor obtained from Admin.
> [~tedyu], want to take a stab at it?
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
