[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-28 Thread stack (JIRA)

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

stack commented on HBASE-13214:
---

HBASE-13983 is resolution of the discussion above around the oddball methods 
getStartKeys, etc. (after a trip to the dev list for discussion under heading 
Semantic Versioning Worksheet.

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-28 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Thanks Stack for all the support.

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Thanks again for looking into this.
[~stack]
{quote}
I don't think you can purge getStartKeys In 1.0 branch I don't see it 
deprecated (it is deprecated in branch-1.1). Maybe I am reading it wrong – I am 
not seeing how they were deprecated in 1.0. Ditto getEndKeys ... and 
getStartEndKeys.
{quote}
No, you are reading it correct these methods were deprecated in 1.1. But what I 
have understood from the below point taken from Semantic Versioning is 
{noformat}
issue a new minor release with the deprecation in place. Before you completely 
remove the functionality in a new major release there should be at least one 
minor release that contains the deprecation so that users can smoothly 
transition to the new API.
{noformat}
We had it deprecated in 1.2, 1.3 so we can remove it in 2.0 right ?

Also HTable is annotated with @InterfaceAudience.Private and from hbase book I 
understood that we can safely remove it
{noformat}
@InterfaceAudience.Private
APIs for HBase internals developers. No guarantees on compatibility or 
availability in future versions. Private interfaces do not need an 
@InterfaceStability classification.
{noformat}
Correct me if I am wrong with my understanding on this ?

{quote}
batch( is not @deprecated in 1.0 if I read it right. Same for exists(  oh, 
nvm, I see deprecated in the Interface. That is fine.
{quote}
I see them deprecated in 1.0 HTable code also. But anyways as you said it is 
deprecated in interface also so I removed it from Table class also.

{quote}
Where is below deprecated:
public Object[] batch(List? extends Row actions) throws IOException {
{quote}
This was deprecated with HBASE-10347 and is available in 1.0 also.

bq. Very nice cleanup.
Thanks

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Rebased patch after HBASE-13893.

 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)


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Thanks all.
Yes, test failures are not related, I checked #6611 HBase-TRUNK job and the 
tests have passed.
{noformat}
Running org.apache.hadoop.hbase.regionserver.TestKeepDeletes
Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.001 sec - in 
org.apache.hadoop.hbase.regionserver.TestKeepDeletes
{noformat}

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

bq. Object[] batch(List? extends Row actions)
It is already deprecated in branch-1.0 code. 
https://github.com/apache/hbase/blob/branch-1.0/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HTable.java

For the rest I will provide a v5 patch which preserves it.

 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)


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

I checked all the deprecated methods removed from HTable class in v5 patch are 
deprecated in branch-1.0 HTable class.

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-13214:
--

200k down to 60k? You sure? Must be a lot of duplicated effort :(

 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)


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Yes even I had to cross check the patch twice whether I am not missing anything.
Did not notice HBASE-13893 before, my bad. It has removed all the HTable 
reference from the test code. So test code modifications has been reduced to a 
great extent. 

 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)


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread stack (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-13214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=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)


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-13214:
--

So [~stack] to summarize you'd like to see a new patch that preserves 
{{getStartKeys}}, {{getEndKeys}}, {{getStartEndKeys}}, and {{Object[] 
batch(List? extends Row actions)}} ?

 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)


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

bq. 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.

Yeah, I think taking it out to separate jira would be good, so this patch 
doesn't depend on whether or not these several methods could be removed.

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12742188/HBASE-13214-v5.patch
  against master branch at commit 0f9c31796850192d2e604036c991e5a607ae4c2e.
  ATTACHMENT ID: 12742188

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 36 new 
or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.0 2.4.1 2.5.0 2.5.1 2.5.2 2.6.0 2.7.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn post-site goal succeeds with this patch.

{color:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14589//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14589//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14589//artifact/patchprocess/checkstyle-aggregate.html

  Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14589//console

This message is automatically generated.

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12742156/HBASE-13214-v4.patch
  against master branch at commit 0f9c31796850192d2e604036c991e5a607ae4c2e.
  ATTACHMENT ID: 12742156

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 36 new 
or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.0 2.4.1 2.5.0 2.5.1 2.5.2 2.6.0 2.7.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn post-site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
 

 {color:red}-1 core zombie tests{color}.  There are 5 zombie test(s):   
at 
org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormatTestBase.testWithMapReduce(TableSnapshotInputFormatTestBase.java:114)
at 
org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormatTestBase.testWithMapReduceAndOfflineHBaseMultiRegion(TableSnapshotInputFormatTestBase.java:101)
at 
org.apache.hadoop.hbase.mapreduce.TestImportExport.testWithDeletes(TestImportExport.java:324)
at 
org.apache.hadoop.hbase.mapreduce.TestRowCounter.testRowCounterHiddenColumn(TestRowCounter.java:143)

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14587//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14587//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14587//artifact/patchprocess/checkstyle-aggregate.html

  Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14587//console

This message is automatically generated.

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Hudson (JIRA)

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

Hudson commented on HBASE-13214:


FAILURE: Integrated in HBase-TRUNK #6610 (See 
[https://builds.apache.org/job/HBase-TRUNK/6610/])
HBASE-13214 Remove deprecated and unused methods from HTable class (ndimiduk: 
rev 3efde85614e80967557a57b1461b22b101d8a946)
* hbase-server/src/main/java/org/apache/hadoop/hbase/client/HTableWrapper.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/client/HTableInterface.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSideNoCodec.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestHTableWrapper.java
* hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/client/RemoteHTable.java
* 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/BufferedMutatorImpl.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestOpenTableInCoprocessor.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestSizeFailures.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/client/Table.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestRpcControllerFactory.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/MetaTableAccessor.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/client/BufferedMutator.java
* 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java
* hbase-server/src/main/resources/hbase-webapps/master/table.jsp
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSink.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/client/HTable.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMultiParallel.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHCM.java


 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-13214:
--

Test failure seems unrelated. Indeed, it does not reproduce locally.

{noformat}
java.lang.NoClassDefFoundError: org/apache/htrace/Trace
at java.net.URLClassLoader$1.run(URLClassLoader.java:366)
at java.net.URLClassLoader$1.run(URLClassLoader.java:355)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:354)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
at 
org.apache.hadoop.hbase.regionserver.wal.FSHLog.rollWriter(FSHLog.java:664)
at 
org.apache.hadoop.hbase.regionserver.wal.FSHLog.rollWriter(FSHLog.java:580)
at 
org.apache.hadoop.hbase.regionserver.wal.FSHLog.init(FSHLog.java:517)
at 
org.apache.hadoop.hbase.wal.DefaultWALProvider.init(DefaultWALProvider.java:94)
at 
org.apache.hadoop.hbase.wal.WALFactory.getProvider(WALFactory.java:146)
at org.apache.hadoop.hbase.wal.WALFactory.init(WALFactory.java:178)
at 
org.apache.hadoop.hbase.HBaseTestingUtility.createWal(HBaseTestingUtility.java:2501)
at 
org.apache.hadoop.hbase.HBaseTestingUtility.createRegionAndWAL(HBaseTestingUtility.java:2523)
at 
org.apache.hadoop.hbase.HBaseTestingUtility.createRegionAndWAL(HBaseTestingUtility.java:2513)
at 
org.apache.hadoop.hbase.HBaseTestingUtility.createLocalHRegion(HBaseTestingUtility.java:1962)
at 
org.apache.hadoop.hbase.HBaseTestingUtility.createLocalHRegion(HBaseTestingUtility.java:1954)
at 
org.apache.hadoop.hbase.regionserver.TestKeepDeletes.testRawScanWithColumns(TestKeepDeletes.java:288)
{noformat}

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-26 Thread stack (JIRA)

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

stack commented on HBASE-13214:
---

[~ashish singhi] thanks.  +1 on v5

 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-v5.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-25 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Any more comments ? If not can we commit this I am worried about it getting 
rotten.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-25 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-13214:
--

Good call on the jruby scripts, thanks for looking into those. Looks like we 
already got a bit of rot, I think from HBASE-13893. Bad luck!

Let's make this happen [~ashish singhi]. Mind taking on the rebase, looks like 
about 25 tests. I'll get you committed post-haste!

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-25 Thread stack (JIRA)

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

stack commented on HBASE-13214:
---

Patch looks great.

I don't think you can purge getStartKeys In 1.0 branch I don't see it 
deprecated (it is deprecated in branch-1.1). Maybe I am reading it wrong -- I 
am not seeing how they were deprecated in 1.0.  Ditto getEndKeys ... and 
getStartEndKeys.

batch( is not @deprecated in 1.0 if I read it right.  Same for exists(  oh, 
nvm, I see deprecated in the Interface. That is fine.

Where is below deprecated:

public Object[] batch(List? extends Row actions) throws IOException {

Very nice cleanup.


 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-25 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

bq. will file one soon.
Created HBASE-13968.

bq. Will comment back if I found anything or not
Checked the ruby scripts did not find any usage for the removed methods.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-24 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

[~anoop.hbase] do you have any more comments ?

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-24 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12741690/HBASE-13214-v3.patch
  against master branch at commit 578adca6ee961f558cd2b2246156f9822cf4f7a2.
  ATTACHMENT ID: 12741690

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 126 
new or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.1 2.5.2 2.6.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn post-site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
 

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14553//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14553//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14553//artifact/patchprocess/checkstyle-aggregate.html

  Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14553//console

This message is automatically generated.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-24 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-13214:
--

Went through the patch, nice bit of cleanup. I'd feel better with a clean 
buildbot run, but those seem to be hard to come by these days. Let's see how 
this next one goes, and please [~ashish singhi] do keep an eye on them after, 
but yeah, looks good to me. +1

[~stack] [~enis] [~sduskis] you folks care to weigh in 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-24 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

Went thru comments on RB (thanks [~anoop.hbase]!) and replies to them, last 
version of patch, looks good to me too. +1.

Let's link in here additional jiras which were created based on comments. From 
RB comments: 

bq. This can be removed? BufferedMutator is @since 1.0.0

Was jira filed for that? Looks like consensus was to create one.


 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-24 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Thanks for the reviews.
bq. do keep an eye on them after
Ya sure.

bq. Was jira filed for that?
Sorry, I missed it will file one soon.

This time the build bot failed for hbase-examples project, so almost all UTs 
related to code change would have been executed.
{quote}
[INFO] Reactor Summary:
[INFO] 
[INFO] HBase . SUCCESS [  2.869 s]
[INFO] HBase - Checkstyle  SUCCESS [  0.509 s]
[INFO] HBase - Annotations ... SUCCESS [  0.754 s]
[INFO] HBase - Protocol .. SUCCESS [ 12.314 s]
[INFO] HBase - Common  SUCCESS [01:24 min]
[INFO] HBase - Procedure . SUCCESS [02:32 min]
[INFO] HBase - Client  SUCCESS [01:23 min]
[INFO] HBase - Hadoop Compatibility .. SUCCESS [  6.963 s]
[INFO] HBase - Hadoop Two Compatibility .. SUCCESS [  6.756 s]
[INFO] HBase - Prefix Tree ... SUCCESS [  9.546 s]
[INFO] HBase - Server  SUCCESS [  01:27 h]
[INFO] HBase - Testing Util .. SUCCESS [  1.037 s]
[INFO] HBase - Thrift  SUCCESS [03:07 min]
[INFO] HBase - Shell . SUCCESS [02:46 min]
[INFO] HBase - Integration Tests . SUCCESS [  1.160 s]
[INFO] HBase - Examples .. FAILURE [  9.330 s]
[INFO] HBase - Rest .. SKIPPED
[INFO] HBase - Assembly .. SKIPPED
[INFO] HBase - Shaded  SKIPPED
[INFO] HBase - Shaded - Client ... SKIPPED
[INFO] HBase - Shaded - Server ... SKIPPED
[INFO] 
[INFO] BUILD FAILURE
[INFO] 
{quote}

Received a good comment offline from [~anoop.hbase] to see if removed methods 
where used in any of the ruby scripts. I missed that part before looking it 
now. Will comment back if I found anything or not.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-22 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

{noformat}
Results :

Failed tests: 
  TestDistributedLogSplitting.testLogReplayTwoSequentialRSDown:651 
expected:1000 but was:939
{noformat}
Not related to this patch.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-20 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Yes failures are unrelated I have seen in other QA runs as well.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-20 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  
http://issues.apache.org/jira/secure/attachment/12740798/HBASE-13214-v2-again-v1.patch
  against master branch at commit db08013ebeeaa85802d9795cc72b4c29c5338a47.
  ATTACHMENT ID: 12740798

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 114 
new or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.1 2.5.2 2.6.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn post-site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
 

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14481//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14481//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14481//artifact/patchprocess/checkstyle-aggregate.html

  Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14481//console

This message is automatically generated.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-20 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Thanks for the review [~anoop.hbase].
Attached v3 patch addressing some review comments from RB.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-20 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12740873/HBASE-13214-v3.patch
  against master branch at commit 04c25e0f355aaa6ded37b0477ce126a693756b81.
  ATTACHMENT ID: 12740873

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 126 
new or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.1 2.5.2 2.6.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

  {color:green}+1 site{color}.  The mvn post-site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
 

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14487//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14487//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14487//artifact/patchprocess/checkstyle-aggregate.html

  Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14487//console

This message is automatically generated.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-19 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  
http://issues.apache.org/jira/secure/attachment/12740587/HBASE-13214-v2-again.patch
  against master branch at commit 8ae4b374e3fa4cd54b8b83bc857d41f0678ad9b6.
  ATTACHMENT ID: 12740587

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 114 
new or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.1 2.5.2 2.6.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

{color:red}-1 site{color}.  The patch appears to cause mvn post-site goal 
to fail.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.mapreduce.TestImportExport
  org.apache.hadoop.hbase.util.TestProcessBasedCluster

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14468//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14468//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14468//artifact/patchprocess/checkstyle-aggregate.html

  Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14468//console

This message is automatically generated.

 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.patch, 
 HBASE-13214-v2.patch, HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-19 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

All failures look unrelated to me. Retry again?

Once we have good hadoop-qa run, v2 patch looks good to me for master. Pinging 
[~stack], [~ndimiduk]?

 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.patch, 
 HBASE-13214-v2.patch, HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-19 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Attached v2 patch again to see if jenkins gets kicked atleast now.
Here is RB link https://reviews.apache.org/r/35642/

 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.patch, 
 HBASE-13214-v2.patch, HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-13214:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12740341/HBASE-13214-v1.patch
  against master branch at commit 51b606cd185437802f0a7a4620f1434e8e2d9c74.
  ATTACHMENT ID: 12740341

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 114 
new or modified tests.

{color:green}+1 hadoop versions{color}. The patch compiles with all 
supported hadoop versions (2.4.1 2.5.2 2.6.0)

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 protoc{color}.  The applied patch does not increase the 
total number of protoc compiler warnings.

{color:red}-1 javadoc{color}.  The javadoc tool appears to have generated 2 
warning messages.

{color:green}+1 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{color:green}+1 findbugs{color}.  The patch does not introduce any  new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:red}-1 lineLengths{color}.  The patch introduces the following lines 
longer than 100:
+try (Table meta = 
connection.getTable(TableName.META_TABLE_NAME, tableExecutorService)) {
+  assertEquals(r.getStartKeys().length, SPLITS.length + 1 + 1); //SPLITS + 
1 is # regions pre-split.
+  assertEquals(r.getStartKeys().length, SPLITS.length + 1 + 1); //SPLITS + 
1 is # regions pre-split.

  {color:green}+1 site{color}.  The mvn post-site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
 

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14457//testReport/
Release Findbugs (version 2.0.3)warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14457//artifact/patchprocess/newFindbugsWarnings.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14457//artifact/patchprocess/checkstyle-aggregate.html

  Javadoc warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14457//artifact/patchprocess/patchJavadocWarnings.txt
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/14457//console

This message is automatically generated.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Attached v2 patch to address 1 javadoc warning related to patch and line 
lengths issue.

 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.patch, 
 HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Tried RB again.. but no luck :(

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

[~ashish singhi] retry? hadoop-qa seems to be slow to response..

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Thanks [~mantonov] for the feedback.
Attached v1 patch which removes unused imports but did not find any unnecessary 
initializations in TestAdmin1.

{quote}
Regarding RB upload - how do you produce patch? I usually commit it locally, 
then do 'git format-patch HEAD^..HEAD --stdout  patch.name', and then just do 
soft/hard reset as needed
{quote}
Same as above.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

Thanks - I kicked in build manually, let's see how it goes.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-18 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Looks like build you triggered #14453 and #14454 has been failed for some weird 
reasons.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-17 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

Good cleanup! Good to see HTable getting 400 lines shorter as result :)

Do we not need this? I kind of remember discussion ([~larsgeorge]?) about 
accessing internal write buffer. Will try to find, but seems we can just remove 
it.

{code}
/**
   * Kept in 0.96 for backward compatibility
   * @deprecated  since 0.96. This is an internal buffer that should not be 
read nor write.
   */
  @Deprecated
  public ListRow getWriteBuffer() {
return mutator == null ? null : mutator.getWriteBuffer();
  }
{code}

Now it looks like there's bunch of unused/deprecated method in HTableInterface, 
but this class is going (I believe) to be deleted anyway, so no need to fix 
those.

Small nits - there's some unused imports in OfflineMetaRebuildTestCore, 
TestMasterObserver, some unnecessary initializations in TestAdmin1.

Looks good otherwise. Let's see what hadoop-qa says. Regarding RB upload - how 
do you produce patch? I usually commit it locally, then do 'git format-patch 
HEAD^..HEAD --stdout  patch.name', and then just do soft/hard reset as needed.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-17 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

I am not able to submit patch to the review board due to the following error
{noformat}
The uploaded diff uses short revisions, but Review Board requires full 
revisions.
Please generate a new diff using the --full-index parameter.
{noformat}

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-17 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Removed all the deprecated and unused methods from the HTable class and 
modified their usage in tests class accordingly.
Verified the UI on the cluster for the modification done in table.jsp file.
Please review.

 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.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-17 Thread Ashish Singhi (JIRA)

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

Ashish Singhi commented on HBASE-13214:
---

Pleasure is all mine! :)

 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
 Attachments: HBASE-13214.patch


 Methods like #getRegionLocation(), #isTableEnabled() etc.



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


[jira] [Commented] (HBASE-13214) Remove deprecated and unused methods from HTable class

2015-06-16 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-13214:
-

Thanks for picking this one up!

 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

 Methods like #getRegionLocation(), #isTableEnabled() etc.



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