[HBASE-18528] DON'T allow user to modify the passed table/column descriptor

• Type: Sub-task
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: Coprocessors, master
• Hadoop Flags: Incompatible change, Reviewed
• Assignee: Chia-Ping Tsai
• Reporter: Chia-Ping Tsai
• Created: 07/Aug/17 04:46
• Updated: 17/Aug/17 19:27
• Resolved: 14/Aug/17 06:11

Description

We are replacing the HTableDescriptor by TableDescriptor from code base. The TableDescriptor is designed to be a read-only object so user can't modifiy it through MasterObserver. HBASE-18502 change many methods of MasterObserver to use TableDescriptor but some deprecated methods still accept the HTableDescriptor. User may be confused by why some methods can't modify the table descriptor.

In short, Should we allow user to modify the passed table descriptor?

1. if yes, we should introduce a mechanism that user can return a modified table descripror
2. if no, we should pass ImmutableHTableDescriptor to user. Or we just remove all methods accepting the HTableDescriptor

Ditto for HColumnDescriptor.

Issue Links

• links to: Review board