[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-20 Thread Hudson (JIRA)

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

Hudson commented on HBASE-9988:
---

SUCCESS: Integrated in hbase-0.96-hadoop2 #125 (See 
[https://builds.apache.org/job/hbase-0.96-hadoop2/125/])
HBASE-9988 DOn't use HRI#getEncodedName in the client (nkeywal: rev 1543428)
* 
/hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
* 
/hbase/branches/0.96/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java


> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-19 Thread Hudson (JIRA)

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

Hudson commented on HBASE-9988:
---

SUCCESS: Integrated in HBase-TRUNK #4688 (See 
[https://builds.apache.org/job/HBase-TRUNK/4688/])
HBASE-9988 DOn't use HRI#getEncodedName in the client (nkeywal: rev 1543427)
* 
/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
* 
/hbase/trunk/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java


> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-19 Thread Hudson (JIRA)

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

Hudson commented on HBASE-9988:
---

SUCCESS: Integrated in hbase-0.96 #196 (See 
[https://builds.apache.org/job/hbase-0.96/196/])
HBASE-9988 DOn't use HRI#getEncodedName in the client (nkeywal: rev 1543428)
* 
/hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
* 
/hbase/branches/0.96/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java


> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-19 Thread Hudson (JIRA)

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

Hudson commented on HBASE-9988:
---

SUCCESS: Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #844 (See 
[https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/844/])
HBASE-9988 DOn't use HRI#getEncodedName in the client (nkeywal: rev 1543427)
* 
/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
* 
/hbase/trunk/hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java


> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-18 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-9988:
-

Thanks for clarifying. +1

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-18 Thread Nicolas Liochon (JIRA)

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

Nicolas Liochon commented on HBASE-9988:


We were just putting the failure in an internal array, without publishing it 
anywhere. As well it was only if log.trace was on, so not really useful in 
practice. We now log the errors (at the info level) after a configurable number 
of retries.

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-18 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-9988:
-

The change in retry logic -- you're no longer gathering the failures? That 
feature is being dropped?

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-18 Thread Nicolas Liochon (JIRA)

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

Nicolas Liochon commented on HBASE-9988:


The patch changed, so I don't mind another review if you have time, Nick :-).
I've removed something used only in "trace" mode, and fixed the previous issue.

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-18 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9988:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12614360/9988.v2.patch
  against trunk revision .

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

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

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

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

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

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) 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 site goal to 
fail.

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

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7911//console

This message is automatically generated.

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch, 9988.v2.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-18 Thread Nicolas Liochon (JIRA)

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

Nicolas Liochon commented on HBASE-9988:


bq. Is the test failure related?
Yes, I need to fix it :-)

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-17 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-9988:
-

+1

Is the test failure related?

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Commented] (HBASE-9988) DOn't use HRI#getEncodedName in the client

2013-11-17 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-9988:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12614283/9988.v1.patch
  against trunk revision .

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

{color:red}-1 tests included{color}.  The patch doesn't appear to include 
any new or modified tests.
Please justify why no new tests are needed for this 
patch.
Also please list what manual steps were performed to 
verify this patch.

{color:green}+1 hadoop1.0{color}.  The patch compiles against the hadoop 
1.0 profile.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

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

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

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) 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 site goal to 
fail.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.client.TestAsyncProcess

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/7904//console

This message is automatically generated.

> DOn't use HRI#getEncodedName in the client
> --
>
> Key: HBASE-9988
> URL: https://issues.apache.org/jira/browse/HBASE-9988
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Affects Versions: 0.98.0, 0.96.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Fix For: 0.98.0, 0.96.1
>
> Attachments: 9988.v1.patch
>
>
> This functions does a lazy initialisation. It cost memory and it creates a 
> synchronisation point.



--
This message was sent by Atlassian JIRA
(v6.1#6144)