[ 
https://issues.apache.org/jira/browse/HBASE-13214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14603282#comment-14603282
 ] 

stack commented on HBASE-13214:
-------------------------------

[~ashish singhi] You make a strong argument but I don't think we should do it. 
I think we need to be more conservative than what semantic versioning says, 
that a minor version with deprecations is sufficient.  Maybe that'll work at 
some time in the future after we have been practicing semantic versioning a 
while but for now I think we need to be more conservative. Let me solicit 
opinion of others out on dev. Also, HTable is annotated private and as you say 
this should give devs free reign making changes but this was our most public 
class just before the 1.0.0 rollout so for it I think we need to go about 
changes with some extra senstivity.

I suggest that we make a separate issue for the removal of getStartKeys et al. 
so they don't hold up the commit of this nice cleanup of yours.

That is kinda sweet that the [~jurmous] HTable cleanup helped here.

> Remove deprecated and unused methods from HTable class
> ------------------------------------------------------
>
>                 Key: HBASE-13214
>                 URL: https://issues.apache.org/jira/browse/HBASE-13214
>             Project: HBase
>          Issue Type: Sub-task
>          Components: API
>    Affects Versions: 2.0.0
>            Reporter: Mikhail Antonov
>            Assignee: Ashish Singhi
>             Fix For: 2.0.0
>
>         Attachments: HBASE-13214-v1.patch, HBASE-13214-v2-again-v1.patch, 
> HBASE-13214-v2-again.patch, HBASE-13214-v2.patch, HBASE-13214-v3.patch, 
> HBASE-13214-v3.patch, HBASE-13214-v4.patch, HBASE-13214.patch
>
>
> Methods like #getRegionLocation(), #isTableEnabled() etc.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to