[jira] [Commented] (HBASE-1403) Move transactional and indexer hbase out of core into contrib or out to their own project

2014-08-06 Thread Liang Li (JIRA)

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

Liang Li commented on HBASE-1403:
-

Hi,stack, will you please tell me the detail reason why you move the index 
down? Also, the current version is 0.98.1, Do you consider to recover the index 
part to main version?

> Move transactional and indexer hbase out of core into contrib or out to their 
> own project
> -
>
> Key: HBASE-1403
> URL: https://issues.apache.org/jira/browse/HBASE-1403
> Project: HBase
>  Issue Type: Task
>Reporter: stack
> Fix For: 0.20.0
>
>
> Its a bit of work bringing along these hbase subclasses as changes happen in 
> core.  They are substantial enough contributions, they could be their own 
> projects -- if there was the will to keep them up.  Otherwise, we could move 
> them down into contrib. -- that'd better demark core and this hbase 
> customizations.
> I wrote Clint asking what he thought.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Assigned] (HBASE-11683) Metrics for MOB

2014-08-06 Thread Jingcheng Du (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11683?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jingcheng Du reassigned HBASE-11683:


Assignee: Jingcheng Du

> Metrics for MOB
> ---
>
> Key: HBASE-11683
> URL: https://issues.apache.org/jira/browse/HBASE-11683
> Project: HBase
>  Issue Type: Sub-task
>  Components: regionserver, Scanners
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Jingcheng Du
>
> We need to make sure to capture metrics about mobs.
> Some basic ones include:
> # of mob writes
> # of mob reads
> # avg size of mob (?)
> # mob files
> # of mob compactions / sweeps



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11683) Metrics for MOB

2014-08-06 Thread Jingcheng Du (JIRA)

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

Jingcheng Du commented on HBASE-11683:
--

Currently we have defined the metrics on the mob flusher.
1. how many times of flushing occurs.
2. how many cells are flushed to the mob file.
3. how many size of mob cells are flushed to the mob file.

about #5 The compaction/sweeps is a tool out of hbase and uses the mapreduce. 
We use the context counter to do the metrics work.
I'm thinking how to implement the #2 mob reads, is it okay to record how many 
times the scanner read from the mob files? I don't see HBase has metrics in the 
normal scanner, is it necessary for the mob read? Please advise. Thanks.

> Metrics for MOB
> ---
>
> Key: HBASE-11683
> URL: https://issues.apache.org/jira/browse/HBASE-11683
> Project: HBase
>  Issue Type: Sub-task
>  Components: regionserver, Scanners
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Jingcheng Du
>
> We need to make sure to capture metrics about mobs.
> Some basic ones include:
> # of mob writes
> # of mob reads
> # avg size of mob (?)
> # mob files
> # of mob compactions / sweeps



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Created] (HBASE-11685) Incr/decr on the reference count of HConnectionImplementation need be atomic

2014-08-06 Thread Liu Shaohui (JIRA)
Liu Shaohui created HBASE-11685:
---

 Summary: Incr/decr on the reference count of 
HConnectionImplementation need be atomic 
 Key: HBASE-11685
 URL: https://issues.apache.org/jira/browse/HBASE-11685
 Project: HBase
  Issue Type: Bug
  Components: Client
Reporter: Liu Shaohui
Assignee: Liu Shaohui
Priority: Minor


Currently, the incr/decr operation on the ref count of 
HConnectionImplementation are not atomic. This may cause that the ref count 
always be larger than 0 and  the connection never be closed.

{code}
/**
 * Increment this client's reference count.
 */
void incCount() {
  ++refCount;
}

/**
 * Decrement this client's reference count.
 */
void decCount() {
  if (refCount > 0) {
--refCount;
  }
}
{code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11438) [Visibility Controller] Support UTF8 character as Visibility Labels

2014-08-06 Thread ramkrishna.s.vasudevan (JIRA)

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

ramkrishna.s.vasudevan commented on HBASE-11438:


bq.But we should validate (like we did in past)
Then the validation should check for double quotes and allow that.  But I don't 
feel that would be needed. Because then we should actually parse the string 
rewrite the string without quotes. Same applies with authorizations also.  
Other argument I would suggest is that if we are allowing everything in the 
addLabels and authorizations what is there to be validated?  And the point to 
see here is that in the addlabels and authorizations there is no expression 
that we are trying to build.  So for using &,!,| should be made simple IMO.

> [Visibility Controller] Support UTF8 character as Visibility Labels
> ---
>
> Key: HBASE-11438
> URL: https://issues.apache.org/jira/browse/HBASE-11438
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Affects Versions: 0.98.4
>Reporter: ramkrishna.s.vasudevan
>Assignee: ramkrishna.s.vasudevan
> Fix For: 0.98.6
>
> Attachments: HBASE-11438_v1.patch, HBASE-11438_v2.patch, 
> HBASE-11438_v3.patch, HBASE-11438_v4.patch, HBASE-11438_v5.patch, 
> HBASE-11438_v6.patch, HBASE-11438_v7.patch
>
>
> This would be an action item that we would be addressing so that the 
> visibility labels could have UTF8 characters in them.  Also allow the user to 
> use a client supplied API that allows to specify the visibility labels inside 
> double quotes such that UTF8 characters and cases like &, |, ! and double 
> quotes itself could be specified with proper escape sequence.  Accumulo too 
> provides one such API in the client side.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11685) Incr/decr on the reference count of HConnectionImplementation need be atomic

2014-08-06 Thread Liu Shaohui (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11685?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Liu Shaohui updated HBASE-11685:


Attachment: HBASE-11685-trunk-v1.diff

Change the refCount from int to AtomicInteger

> Incr/decr on the reference count of HConnectionImplementation need be atomic 
> -
>
> Key: HBASE-11685
> URL: https://issues.apache.org/jira/browse/HBASE-11685
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Reporter: Liu Shaohui
>Assignee: Liu Shaohui
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-11685-trunk-v1.diff
>
>
> Currently, the incr/decr operation on the ref count of 
> HConnectionImplementation are not atomic. This may cause that the ref count 
> always be larger than 0 and  the connection never be closed.
> {code}
> /**
>  * Increment this client's reference count.
>  */
> void incCount() {
>   ++refCount;
> }
> /**
>  * Decrement this client's reference count.
>  */
> void decCount() {
>   if (refCount > 0) {
> --refCount;
>   }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11685) Incr/decr on the reference count of HConnectionImplementation need be atomic

2014-08-06 Thread Liu Shaohui (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11685?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Liu Shaohui updated HBASE-11685:


Fix Version/s: 2.0.0
   Status: Patch Available  (was: Open)

> Incr/decr on the reference count of HConnectionImplementation need be atomic 
> -
>
> Key: HBASE-11685
> URL: https://issues.apache.org/jira/browse/HBASE-11685
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Reporter: Liu Shaohui
>Assignee: Liu Shaohui
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-11685-trunk-v1.diff
>
>
> Currently, the incr/decr operation on the ref count of 
> HConnectionImplementation are not atomic. This may cause that the ref count 
> always be larger than 0 and  the connection never be closed.
> {code}
> /**
>  * Increment this client's reference count.
>  */
> void incCount() {
>   ++refCount;
> }
> /**
>  * Decrement this client's reference count.
>  */
> void decCount() {
>   if (refCount > 0) {
> --refCount;
>   }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11438) [Visibility Controller] Support UTF8 character as Visibility Labels

2014-08-06 Thread Anoop Sam John (JIRA)

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

Anoop Sam John commented on HBASE-11438:


So you plan to undo HBASE-10883 fix?

> [Visibility Controller] Support UTF8 character as Visibility Labels
> ---
>
> Key: HBASE-11438
> URL: https://issues.apache.org/jira/browse/HBASE-11438
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Affects Versions: 0.98.4
>Reporter: ramkrishna.s.vasudevan
>Assignee: ramkrishna.s.vasudevan
> Fix For: 0.98.6
>
> Attachments: HBASE-11438_v1.patch, HBASE-11438_v2.patch, 
> HBASE-11438_v3.patch, HBASE-11438_v4.patch, HBASE-11438_v5.patch, 
> HBASE-11438_v6.patch, HBASE-11438_v7.patch
>
>
> This would be an action item that we would be addressing so that the 
> visibility labels could have UTF8 characters in them.  Also allow the user to 
> use a client supplied API that allows to specify the visibility labels inside 
> double quotes such that UTF8 characters and cases like &, |, ! and double 
> quotes itself could be specified with proper escape sequence.  Accumulo too 
> provides one such API in the client side.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11685) Incr/decr on the reference count of HConnectionImplementation need be atomic

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11685:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  
http://issues.apache.org/jira/secure/attachment/12660120/HBASE-11685-trunk-v1.diff
  against trunk revision .
  ATTACHMENT ID: 12660120

{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 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

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

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

{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 site goal succeeds with this patch.

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

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

This message is automatically generated.

> Incr/decr on the reference count of HConnectionImplementation need be atomic 
> -
>
> Key: HBASE-11685
> URL: https://issues.apache.org/jira/browse/HBASE-11685
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Reporter: Liu Shaohui
>Assignee: Liu Shaohui
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-11685-trunk-v1.diff
>
>
> Currently, the incr/decr operation on the ref count of 
> HConnectionImplementation are not atomic. This may cause that the ref count 
> always be larger than 0 and  the connection never be closed.
> {code}
> /**
>  * Increment this client's reference count.
>  */
> void incCount() {
>   ++refCount;
> }
> /**
>  * Decrement this client's reference count.
>  */
> void decCount() {
>   if (refCount > 0) {
> --refCount;
>   }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11685) Incr/decr on the reference count of HConnectionImplementation need be atomic

2014-08-06 Thread Liu Shaohui (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11685?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Liu Shaohui updated HBASE-11685:


Attachment: HBASE-11685-trunk-v2.diff

Fix failed test: TestClientNoCluster

> Incr/decr on the reference count of HConnectionImplementation need be atomic 
> -
>
> Key: HBASE-11685
> URL: https://issues.apache.org/jira/browse/HBASE-11685
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Reporter: Liu Shaohui
>Assignee: Liu Shaohui
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-11685-trunk-v1.diff, HBASE-11685-trunk-v2.diff
>
>
> Currently, the incr/decr operation on the ref count of 
> HConnectionImplementation are not atomic. This may cause that the ref count 
> always be larger than 0 and  the connection never be closed.
> {code}
> /**
>  * Increment this client's reference count.
>  */
> void incCount() {
>   ++refCount;
> }
> /**
>  * Decrement this client's reference count.
>  */
> void decCount() {
>   if (refCount > 0) {
> --refCount;
>   }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Resolved] (HBASE-9416) Use the disruptor library in the rcpServer

2014-08-06 Thread Nicolas Liochon (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-9416?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nicolas Liochon resolved HBASE-9416.


Resolution: Won't Fix

No work on this for a while, the patch need to be redone totally with the 
recent changes on this part => solving as won't fix.

> Use the disruptor library in the rcpServer
> --
>
> Key: HBASE-9416
> URL: https://issues.apache.org/jira/browse/HBASE-9416
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Affects Versions: 0.98.0
>Reporter: Nicolas Liochon
>Assignee: Nicolas Liochon
> Attachments: disrupt.patch
>
>
> Nasty & hacky patch on top of the 0.96 to get some feedback on adding this 
> third party.
> I ran a test doing "gets" on an empty region.
> With the current implementation, we're spending time in the 
> LinkedBlockingQueue#put. I was able to do 150K operations per second.
> Using the disruptor allowed me to go to 190 ops/s, i.e. a little be more than 
> a 25% improvement.
> Likely there are other improvements in this class as well.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11657) Put HTable region methods in an interface

2014-08-06 Thread Carter (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carter updated HBASE-11657:
---

Status: Open  (was: Patch Available)

> Put HTable region methods in an interface
> -
>
> Key: HBASE-11657
> URL: https://issues.apache.org/jira/browse/HBASE-11657
> Project: HBase
>  Issue Type: Improvement
>Affects Versions: 0.99.0
>Reporter: Carter
>Assignee: Carter
> Fix For: 0.99.0
>
> Attachments: HBASE_11657.patch, HBASE_11657_v2.patch, 
> HBASE_11657_v3.patch
>
>
> Most of the HTable methods are now abstracted by HTableInterface, with the 
> notable exception of the following methods that pertain to region metadata:
> {code}
> HRegionLocation getRegionLocation(final String row)
> HRegionLocation getRegionLocation(final byte [] row)
> HRegionLocation getRegionLocation(final byte [] row, boolean reload)
> byte [][] getStartKeys()
> byte[][] getEndKeys()
> Pair getStartEndKeys()
> void clearRegionCache()
> {code}
> and a default scope method which maybe should be bundled with the others:
> {code}
> List listRegionLocations()
> {code}
> Since the consensus seems to be that these would muddy HTableInterface with 
> non-core functionality, where should it go?  MapReduce looks up the region 
> boundaries, so it needs to be exposed somewhere.
> Let me throw out a straw man to start the conversation.  I propose:
> {code}
> org.apache.hadoop.hbase.client.HRegionInterface
> {code}
> Have HTable implement this interface.  Also add these methods to HConnection:
> {code}
> HRegionInterface getTableRegion(TableName tableName)
> HRegionInterface getTableRegion(TableName tableName, ExecutorService pool)
> {code}
> [~stack], [~ndimiduk], [~enis], thoughts?



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11657) Put HTable region methods in an interface

2014-08-06 Thread Carter (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carter updated HBASE-11657:
---

Attachment: HBASE_11657_v3.patch

> Put HTable region methods in an interface
> -
>
> Key: HBASE-11657
> URL: https://issues.apache.org/jira/browse/HBASE-11657
> Project: HBase
>  Issue Type: Improvement
>Affects Versions: 0.99.0
>Reporter: Carter
>Assignee: Carter
> Fix For: 0.99.0
>
> Attachments: HBASE_11657.patch, HBASE_11657_v2.patch, 
> HBASE_11657_v3.patch
>
>
> Most of the HTable methods are now abstracted by HTableInterface, with the 
> notable exception of the following methods that pertain to region metadata:
> {code}
> HRegionLocation getRegionLocation(final String row)
> HRegionLocation getRegionLocation(final byte [] row)
> HRegionLocation getRegionLocation(final byte [] row, boolean reload)
> byte [][] getStartKeys()
> byte[][] getEndKeys()
> Pair getStartEndKeys()
> void clearRegionCache()
> {code}
> and a default scope method which maybe should be bundled with the others:
> {code}
> List listRegionLocations()
> {code}
> Since the consensus seems to be that these would muddy HTableInterface with 
> non-core functionality, where should it go?  MapReduce looks up the region 
> boundaries, so it needs to be exposed somewhere.
> Let me throw out a straw man to start the conversation.  I propose:
> {code}
> org.apache.hadoop.hbase.client.HRegionInterface
> {code}
> Have HTable implement this interface.  Also add these methods to HConnection:
> {code}
> HRegionInterface getTableRegion(TableName tableName)
> HRegionInterface getTableRegion(TableName tableName, ExecutorService pool)
> {code}
> [~stack], [~ndimiduk], [~enis], thoughts?



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11657) Put HTable region methods in an interface

2014-08-06 Thread Carter (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carter updated HBASE-11657:
---

Status: Patch Available  (was: Open)

Good point.  Stability changed to "Evolving".  Patch v3 submitted.

> Put HTable region methods in an interface
> -
>
> Key: HBASE-11657
> URL: https://issues.apache.org/jira/browse/HBASE-11657
> Project: HBase
>  Issue Type: Improvement
>Affects Versions: 0.99.0
>Reporter: Carter
>Assignee: Carter
> Fix For: 0.99.0
>
> Attachments: HBASE_11657.patch, HBASE_11657_v2.patch, 
> HBASE_11657_v3.patch
>
>
> Most of the HTable methods are now abstracted by HTableInterface, with the 
> notable exception of the following methods that pertain to region metadata:
> {code}
> HRegionLocation getRegionLocation(final String row)
> HRegionLocation getRegionLocation(final byte [] row)
> HRegionLocation getRegionLocation(final byte [] row, boolean reload)
> byte [][] getStartKeys()
> byte[][] getEndKeys()
> Pair getStartEndKeys()
> void clearRegionCache()
> {code}
> and a default scope method which maybe should be bundled with the others:
> {code}
> List listRegionLocations()
> {code}
> Since the consensus seems to be that these would muddy HTableInterface with 
> non-core functionality, where should it go?  MapReduce looks up the region 
> boundaries, so it needs to be exposed somewhere.
> Let me throw out a straw man to start the conversation.  I propose:
> {code}
> org.apache.hadoop.hbase.client.HRegionInterface
> {code}
> Have HTable implement this interface.  Also add these methods to HConnection:
> {code}
> HRegionInterface getTableRegion(TableName tableName)
> HRegionInterface getTableRegion(TableName tableName, ExecutorService pool)
> {code}
> [~stack], [~ndimiduk], [~enis], thoughts?



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jimmy Xiang updated HBASE-11611:


Attachment: hbase-11611.patch

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11606) Enable ZK-less region assignment by default

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11606:
-

I see. I attached a patch for HBASE-11611, with which I have verified that our 
internal jenkins is green. We should keep jenkins ok.

> Enable ZK-less region assignment by default
> ---
>
> Key: HBASE-11606
> URL: https://issues.apache.org/jira/browse/HBASE-11606
> Project: HBase
>  Issue Type: Task
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: hbase-11606.patch
>
>
> Let's enable ZK-less region assignment by default in the master branch.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11611:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660145/hbase-11611.patch
  against trunk revision .
  ATTACHMENT ID: 12660145

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

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

{color:red}-1 patch{color}.  The patch command could not apply the patch.

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

This message is automatically generated.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11611:
-

Patch v1 is on RB: https://reviews.apache.org/r/24382/

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11682) Explain hotspotting

2014-08-06 Thread Jonathan Hsieh (JIRA)

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

Jonathan Hsieh commented on HBASE-11682:


Nice addition.  Personally, I don't really like the sematext definition of 
salting it conflates salting [1] with hashing[2] which are two separate things.

*salting* adds random data to the start of a rowkey. this means depending on 
the 'salt factor' you could end up writing to n different row keys (and ideally 
n different regions).  When reading you would generally want to read all n rows 
and coalesce the values.  This is helpful if you have individual hot keys.  It 
is often a bad smell because it is a trick used to try to mitigate having the 
date as the first part of a row key, but does have valid use cases. (ex: rowkey 
is name and you have a handful of individual celebreties  - obama, bieber, gaga 
- that need to have their load spread).  This preserves ordering but multiplies 
the number of reads required wrt # of writes.

*hashing* applies a random one way function to the rowkey such that a 
particular row will get the same 'random' value prepended.  The original row 
would get mapped to a single row.   This is good for when you have clusters of 
related keys that in aggregate form a hotspot.  (Example: rowkey is name and 
you have way to many joe's, john's, jon's, jonah's, jonathan's, and jonathons's 
all on the same region -- using a hash would spread all the j names around).  
this throws out the ability to effectively take advantage of the row ordering 
properties.

Another trick is to take numeric or fixed length values and make the least 
significant digit (e.g. the one that changes the most) in least significant 
digit order (little endian).  This effectively randomizes row key names but 
also sacrifices row ordering properties.

[1] http://en.wikipedia.org/wiki/Salt_(cryptography)
[2] http://en.wikipedia.org/wiki/Hash_function

> Explain hotspotting
> ---
>
> Key: HBASE-11682
> URL: https://issues.apache.org/jira/browse/HBASE-11682
> Project: HBase
>  Issue Type: Task
>  Components: documentation
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11682-1.patch, HBASE-11682.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jimmy Xiang updated HBASE-11611:


Attachment: hbase-11611_v1.patch

Attached the same patch generated with --no-prefix. Our patch script is changed 
again?

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11611:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660148/hbase-11611_v1.patch
  against trunk revision .
  ATTACHMENT ID: 12660148

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

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

{color:red}-1 patch{color}.  The patch command could not apply the patch.

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

This message is automatically generated.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jimmy Xiang updated HBASE-11611:


Status: Patch Available  (was: Open)

Attached a patch that removed ZK related assignment code, fixed tests.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11611:
-

That's weird. The patch applies locally.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11657) Put HTable region methods in an interface

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11657:
---

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

{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 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

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

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

{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 site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.master.TestRestartCluster
  org.apache.hadoop.hbase.client.TestReplicasClient
  org.apache.hadoop.hbase.regionserver.TestRegionReplicas
  
org.apache.hadoop.hbase.master.TestMasterOperationsForRegionReplicas

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

This message is automatically generated.

> Put HTable region methods in an interface
> -
>
> Key: HBASE-11657
> URL: https://issues.apache.org/jira/browse/HBASE-11657
> Project: HBase
>  Issue Type: Improvement
>Affects Versions: 0.99.0
>Reporter: Carter
>Assignee: Carter
> Fix For: 0.99.0
>
> Attachments: HBASE_11657.patch, HBASE_11657_v2.patch, 
> HBASE_11657_v3.patch
>
>
> Most of the HTable methods are now abstracted by HTableInterface, with the 
> notable exception of the following methods that pertain to region metadata:
> {code}
> HRegionLocation getRegionLocation(final String row)
> HRegionLocation getRegionLocation(final byte [] row)
> HRegionLocation getRegionLocation(final byte [] row, boolean reload)
> byte [][] getStartKeys()
> byte[][] getEndKeys()
> Pair getStartEndKeys()
> void clearRegionCache()
> {code}
> and a default scope method which maybe should be bundled with the others:
> {code}
> List listRegionLocations()
> {code}
> Since the consensus seems to be that these would muddy HTableInterface with 
> non-core functionality, where should it go?  MapReduce looks up the region 
> boundaries, so it needs to be exposed somewhere.
> Let me throw out a straw man to start the conversation.  I propose:
> {code}
> org.apache.hadoop.hbase.client.HRegionInterface
> {code}
> Have HTable implement this interface.  Also add these methods to HConnection:
> {code}
> HRegionInterface getTabl

[jira] [Updated] (HBASE-11553) Abstract visibility label related services into an interface

2014-08-06 Thread Anoop Sam John (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anoop Sam John updated HBASE-11553:
---

Attachment: HBASE-11553_V2.patch

> Abstract visibility label related services into an interface
> 
>
> Key: HBASE-11553
> URL: https://issues.apache.org/jira/browse/HBASE-11553
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Reporter: Anoop Sam John
>Assignee: Anoop Sam John
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11553.patch, HBASE-11553.patch, 
> HBASE-11553_V2.patch
>
>
> - storage and retrieval of label dictionary and authentication sets 
> - marshalling and unmarshalling of visibility expression representations 
> in operation attributes and cell tags
> - management of assignment of authorizations to principals
> This will allow us to introduce additional serde implementations for 
> visibility expressions, for example storing as strings in some places and 
> compressed/tokenized representation in others in order to support additional 
> use cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11553) Abstract visibility label related services into an interface

2014-08-06 Thread Anoop Sam John (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anoop Sam John updated HBASE-11553:
---

Status: Patch Available  (was: Open)

> Abstract visibility label related services into an interface
> 
>
> Key: HBASE-11553
> URL: https://issues.apache.org/jira/browse/HBASE-11553
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Reporter: Anoop Sam John
>Assignee: Anoop Sam John
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11553.patch, HBASE-11553.patch, 
> HBASE-11553_V2.patch
>
>
> - storage and retrieval of label dictionary and authentication sets 
> - marshalling and unmarshalling of visibility expression representations 
> in operation attributes and cell tags
> - management of assignment of authorizations to principals
> This will allow us to introduce additional serde implementations for 
> visibility expressions, for example storing as strings in some places and 
> compressed/tokenized representation in others in order to support additional 
> use cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11553) Abstract visibility label related services into an interface

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11553:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660153/HBASE-11553_V2.patch
  against trunk revision .
  ATTACHMENT ID: 12660153

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

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

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

Compilation errors resume:
[ERROR] COMPILATION ERROR : 
[ERROR] 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/hbase-server/src/main/java/org/apache/hadoop/hbase/mapreduce/LabelExpander.java:[88,28]
 error: cannot find symbol
[ERROR]   symbol:   variable VIS_SERIALIZATION_TAG
[ERROR]   symbol:   variable VISIBILITY_TAG_TYPE
[ERROR]   symbol:   variable VISIBILITY_TAG_TYPE
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile (default-compile) 
on project hbase-server: Compilation failure: Compilation failure:
[ERROR] 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/hbase-server/src/main/java/org/apache/hadoop/hbase/mapreduce/LabelExpander.java:[88,28]
 error: cannot find symbol
[ERROR] symbol:   variable VIS_SERIALIZATION_TAG
[ERROR] location: class VisibilityUtils
[ERROR] 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/hbase-server/src/main/java/org/apache/hadoop/hbase/mapreduce/LabelExpander.java:[92,38]
 error: cannot find symbol
[ERROR] symbol:   variable VISIBILITY_TAG_TYPE
[ERROR] location: class VisibilityUtils
[ERROR] 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/hbase-server/src/main/java/org/apache/hadoop/hbase/mapreduce/LabelExpander.java:[100,42]
 error: cannot find symbol
[ERROR] symbol:   variable VISIBILITY_TAG_TYPE
[ERROR] location: class VisibilityUtils
[ERROR] 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/hbase-server/src/main/java/org/apache/hadoop/hbase/mapreduce/LabelExpander.java:[107,40]
 error: cannot find symbol
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please 
read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :hbase-server


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

This message is automatically generated.

> Abstract visibility label related services into an interface
> 
>
> Key: HBASE-11553
> URL: https://issues.apache.org/jira/browse/HBASE-11553
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Reporter: Anoop Sam John
>Assignee: Anoop Sam John
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11553.patch, HBASE-11553.patch, 
> HBASE-11553_V2.patch
>
>
> - storage and retrieval of label dictionary and authentication sets 
> - marshalling and unmarshalling of visibility expression representations 
> in operation attributes and cell tags
> - management of assignment of authorizations to principals
> This will allow us to introduce additional serde implementations for 
> visibility expressions, for example storing as strings in some places and 
> compressed/tokenized representation in others in order to support additional 
> use cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11438) [Visibility Controller] Support UTF8 character as Visibility Labels

2014-08-06 Thread ramkrishna.s.vasudevan (JIRA)

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

ramkrishna.s.vasudevan commented on HBASE-11438:


bq.So you plan to undo HBASE-10883 fix?
Yes but not fully because the validation is still getting used in checking if 
the visibility expression without quotes has valid characters as specified in 
the VisibilityLabelsValidator.  
If you see the description of HBASE-10883 it is mainly to disallow usage of 
&,!,| type of characters so that users should not use them. So by removing that 
check we are ensuring that user can use them now.  But in authorizations there 
are no expressions and only string literals. 

> [Visibility Controller] Support UTF8 character as Visibility Labels
> ---
>
> Key: HBASE-11438
> URL: https://issues.apache.org/jira/browse/HBASE-11438
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Affects Versions: 0.98.4
>Reporter: ramkrishna.s.vasudevan
>Assignee: ramkrishna.s.vasudevan
> Fix For: 0.98.6
>
> Attachments: HBASE-11438_v1.patch, HBASE-11438_v2.patch, 
> HBASE-11438_v3.patch, HBASE-11438_v4.patch, HBASE-11438_v5.patch, 
> HBASE-11438_v6.patch, HBASE-11438_v7.patch
>
>
> This would be an action item that we would be addressing so that the 
> visibility labels could have UTF8 characters in them.  Also allow the user to 
> use a client supplied API that allows to specify the visibility labels inside 
> double quotes such that UTF8 characters and cases like &, |, ! and double 
> quotes itself could be specified with proper escape sequence.  Accumulo too 
> provides one such API in the client side.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11682) Explain hotspotting

2014-08-06 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-11682:
--

Please note that "salting" as we use the term in HBase is different from 
"salting" in the cryptographic sense. Our usage pattern is more accurately 
described as "bucketing" (I think this is the term Phoenix uses), or "binning" 
[0]. This horse has been beaten to death on the user and dev mailing lists, so 
I won't belabor the point.

[0]: http://en.wikipedia.org/wiki/Data_binning

> Explain hotspotting
> ---
>
> Key: HBASE-11682
> URL: https://issues.apache.org/jira/browse/HBASE-11682
> Project: HBase
>  Issue Type: Task
>  Components: documentation
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11682-1.patch, HBASE-11682.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11682) Explain hotspotting

2014-08-06 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-11682:
--

bq. However, using totally random row keys for data which is accessed 
sequentially would remove the benefit of HBase's row-sorting algorithm and 
cause very poor performance, as each get or scan would need to query all 
regions.

Prefixing with random byte prevents any meaningful use of scans; gets become 
your only option. This approach is indistinguishable from hashing the rowkey.

I like the rest of the updates, +1

Thanks a lot [~misty]!

> Explain hotspotting
> ---
>
> Key: HBASE-11682
> URL: https://issues.apache.org/jira/browse/HBASE-11682
> Project: HBase
>  Issue Type: Task
>  Components: documentation
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11682-1.patch, HBASE-11682.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Mikhail Antonov (JIRA)

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

Mikhail Antonov commented on HBASE-11611:
-

BTW, the patch doesn't apply with either 'git apply -p0' or 'git apply -p1', 
can only apply with Intellij Idea.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11611:
-

Check dev-support/smart-apply-patch.sh. That's the command to apply the patch: 
$PATCH -p0 -E --dry-run < $PATCH_FILE 2>&1 > $TMP
It should be "/usr/bin/patch" instead of "git apply".

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11553) Abstract visibility label related services into an interface

2014-08-06 Thread Anoop Sam John (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anoop Sam John updated HBASE-11553:
---

Attachment: HBASE-11553_V2.patch

> Abstract visibility label related services into an interface
> 
>
> Key: HBASE-11553
> URL: https://issues.apache.org/jira/browse/HBASE-11553
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Reporter: Anoop Sam John
>Assignee: Anoop Sam John
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11553.patch, HBASE-11553.patch, 
> HBASE-11553_V2.patch
>
>
> - storage and retrieval of label dictionary and authentication sets 
> - marshalling and unmarshalling of visibility expression representations 
> in operation attributes and cell tags
> - management of assignment of authorizations to principals
> This will allow us to introduce additional serde implementations for 
> visibility expressions, for example storing as strings in some places and 
> compressed/tokenized representation in others in order to support additional 
> use cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11553) Abstract visibility label related services into an interface

2014-08-06 Thread Anoop Sam John (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anoop Sam John updated HBASE-11553:
---

Attachment: (was: HBASE-11553_V2.patch)

> Abstract visibility label related services into an interface
> 
>
> Key: HBASE-11553
> URL: https://issues.apache.org/jira/browse/HBASE-11553
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Reporter: Anoop Sam John
>Assignee: Anoop Sam John
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11553.patch, HBASE-11553.patch, 
> HBASE-11553_V2.patch
>
>
> - storage and retrieval of label dictionary and authentication sets 
> - marshalling and unmarshalling of visibility expression representations 
> in operation attributes and cell tags
> - management of assignment of authorizations to principals
> This will allow us to introduce additional serde implementations for 
> visibility expressions, for example storing as strings in some places and 
> compressed/tokenized representation in others in order to support additional 
> use cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11678) BucketCache ramCache fills heap after running a few hours

2014-08-06 Thread stack (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-11678:
--

Attachment: gc_over_12_hours_unevenblocks_with_lots_of_evictions.png

Ran for 12 hours. GC plot looks clean. It may be tending up slightly... will 
run for longer next time but looking at heap dump, this ramCache issue is no 
longer the culprit.

Looking to make a unit test. Its a bit awkward given how deep down doDrain is 
inside BucketCache but worth doing I'd say.

> BucketCache ramCache fills heap after running a few hours
> -
>
> Key: HBASE-11678
> URL: https://issues.apache.org/jira/browse/HBASE-11678
> Project: HBase
>  Issue Type: Bug
>  Components: BlockCache
>Affects Versions: 0.99.0, 0.98.5, 2.0.0
>Reporter: stack
>Assignee: stack
>Priority: Critical
> Attachments: 
> 0001-When-we-failed-add-an-entry-failing-with-a-CacheFull.patch, 
> gc_over_12_hours_unevenblocks_with_lots_of_evictions.png
>
>
> Testing BucketCache, my heap filled after running for hours. Dumping heap, 
> culprit is the ramCache Map in BucketCache.  Tried running with more writer 
> threads but made no difference.  Trying to figure now how our accounting is 
> going wonky.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11682) Explain hotspotting

2014-08-06 Thread Jonathan Hsieh (JIRA)

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

Jonathan Hsieh commented on HBASE-11682:


bq. Please note that "salting" as we use the term in HBase is different from 
"salting" in the cryptographic sense. Our usage pattern is more accurately 
described as "bucketing" (I think this is the term Phoenix uses), or "binning" 
[0]. This horse has been beaten to death on the user and dev mailing lists, so 
I won't belabor the point.

I agree they are different but the idea of a random (as opposed to a 
deterministic hash) prepended is very similar to the crypto "salting".  Instead 
of using "salting" how about we use the term "striping"?  What I've was 
referring and described takes one logical row and stripes it across many rows 
so that write throughput can be increased.  The penalty is that we loose some 
consistency and also if we want a "whole" answer read all the rows the logical 
rows was striped over.  I've seen the pattern deployed at several customers.

>From what phoenix uses and what I think you mean, "bucketing', and "binning" 
>are equivalent to prepending a hash.

> Explain hotspotting
> ---
>
> Key: HBASE-11682
> URL: https://issues.apache.org/jira/browse/HBASE-11682
> Project: HBase
>  Issue Type: Task
>  Components: documentation
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11682-1.patch, HBASE-11682.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11678) BucketCache ramCache fills heap after running a few hours

2014-08-06 Thread Andrew Purtell (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Purtell updated HBASE-11678:
---

Fix Version/s: 0.98.6
   2.0.0
   0.99.0

> BucketCache ramCache fills heap after running a few hours
> -
>
> Key: HBASE-11678
> URL: https://issues.apache.org/jira/browse/HBASE-11678
> Project: HBase
>  Issue Type: Bug
>  Components: BlockCache
>Affects Versions: 0.99.0, 0.98.5, 2.0.0
>Reporter: stack
>Assignee: stack
>Priority: Critical
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: 
> 0001-When-we-failed-add-an-entry-failing-with-a-CacheFull.patch, 
> gc_over_12_hours_unevenblocks_with_lots_of_evictions.png
>
>
> Testing BucketCache, my heap filled after running for hours. Dumping heap, 
> culprit is the ramCache Map in BucketCache.  Tried running with more writer 
> threads but made no difference.  Trying to figure now how our accounting is 
> going wonky.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11553) Abstract visibility label related services into an interface

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11553:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660168/HBASE-11553_V2.patch
  against trunk revision .
  ATTACHMENT ID: 12660168

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

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

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

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

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

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 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 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 1 zombie test(s):   
at 
org.apache.hadoop.hbase.http.TestHttpServerLifecycle.testStartedServerIsAlive(TestHttpServerLifecycle.java:72)

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

This message is automatically generated.

> Abstract visibility label related services into an interface
> 
>
> Key: HBASE-11553
> URL: https://issues.apache.org/jira/browse/HBASE-11553
> Project: HBase
>  Issue Type: Improvement
>  Components: security
>Reporter: Anoop Sam John
>Assignee: Anoop Sam John
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11553.patch, HBASE-11553.patch, 
> HBASE-11553_V2.patch
>
>
> - storage and retrieval of label dictionary and authentication sets 
> - marshalling and unmarshalling of visibility expression representations 
> in operation attributes and cell tags
> - management of assignment of authorizations to principals
> This will allow us to introduce additional serde implementations for 
> visibility expressions, for example storing as strings in some places and 
> compressed/tokenized representation in others in order to support additional 
> use cases.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Virag Kothari (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Virag Kothari updated HBASE-11535:
--

Attachment: HBASE-11535_98.patch
HBASE-11535.patch

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Virag Kothari (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Virag Kothari updated HBASE-11535:
--

Attachment: (was: HBASE-11535_98.patch)

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Attachments: HBASE-11535.patch, HBASE-11535.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Virag Kothari (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Virag Kothari updated HBASE-11535:
--

Attachment: HBASE-11535_98.patch

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Virag Kothari (JIRA)

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

Virag Kothari commented on HBASE-11535:
---

Thanks for pointing that [~jmhsieh]. Updated based on your comments and also 
provided a 0.98 patch

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Virag Kothari (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Virag Kothari updated HBASE-11535:
--

Fix Version/s: 0.98.6

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11546) Backport HBASE-11059 to 0.98

2014-08-06 Thread Virag Kothari (JIRA)

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

Virag Kothari commented on HBASE-11546:
---

Yes, end of August sounds good.

> Backport HBASE-11059 to 0.98
> 
>
> Key: HBASE-11546
> URL: https://issues.apache.org/jira/browse/HBASE-11546
> Project: HBase
>  Issue Type: Task
>Reporter: Virag Kothari
>
> Discuss concerns about backporting HBASE-11059 to 0.98



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Alex Newman (JIRA)

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

Alex Newman commented on HBASE-11665:
-

1. [~devaraj] Was that comment meant for this jira

2. Other tests that do this (should I fix them?)

TestDrainingServers
TestAssigningManager
TestAssignmentManagerOnCluster
TestMasterFailover
TestOpenRegionHandler
TestRestartCluster
TestZKOpenRegion
TestZKAMOnCluster

TestNameSpaceUpgrade <- Is still broken?
TestSplitTransactionOnCluster
TestZKLessMerge - sets it to false
TestCloseRegionHandler
TestOpenRegionHandler

ConfigUtil - sets it to false

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 0.99.0, 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11535:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660179/HBASE-11535_98.patch
  against trunk revision .
  ATTACHMENT ID: 12660179

{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:red}-1 patch{color}.  The patch command could not apply the patch.

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

This message is automatically generated.

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11665:
-

Can you try with patch HBASE-11611? These tests are fixed/removed. We don't 
need to fix them one-by-one any more.

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 0.99.0, 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11665:
-

BTW, HBASE-11611 is for master branch only. This test should work for branch 1.

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 0.99.0, 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread Sean Busbey (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11658?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Busbey updated HBASE-11658:


Attachment: HBASE-11658.manual_test.log
test_shell.sh
HBASE-11658.patch

attaching patch so that QA can take a run at it.

Also attaching bash script used for manually verifying behavior and the log 
from a run of it.

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11535:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660177/HBASE-11535.patch
  against trunk revision .
  ATTACHMENT ID: 12660177

{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 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

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

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

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 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 site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.procedure.TestProcedureManager
  org.apache.hadoop.hbase.ipc.TestIPC
  org.apache.hadoop.hbase.master.TestClockSkewDetection

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

This message is automatically generated.

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread Sean Busbey (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11658?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Busbey updated HBASE-11658:


Status: Patch Available  (was: In Progress)

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Comment Edited] (HBASE-11331) [blockcache] lazy block decompression

2014-08-06 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk edited comment on HBASE-11331 at 8/6/14 6:16 PM:
--

re: lazy-decompress.02.0.pdf

Attaching some metrics after running v02 with the new flag enabled/disabled.

The test cluster is a single machine, configured with 12g heap and 0.6 for 
blockcache, so slighly over 7g mem available for cache. The TestTable is snappy 
compressed, written with PE --size=100. Shell {{status 'simple'}} reports 
{{compressionRatio=0.2437}}. RandomRead test is run with --size=25, so with 
feature enabled, we should be at ~6g of compressed blocks.
h3. client perceived perf
With flag=false, I see slightly higher throughput (~7.5k vs ~8k) and slightly 
less gc (360ms vs 325ms).
h3. system load
CPU load looks slightly higher with flag=true (user time diff is negligible, 
system time more noticeable, no difference in iowait).
h3. blockcache utilization
There are definitely cache evictions happening with flag=false that do not 
happen with flag=true. The number of blocks cached is definitely higher with 
flag=true (~17.5k vs ~21.5k).
h3. IO
No difference in reported IO requests. Probably it's the OS buffer cache to the 
rescue here.


was (Author: ndimiduk):
Attaching some metrics after running v02 with the new flag enabled/disabled.

The test cluster is a single machine, configured with 12g heap and 0.6 for 
blockcache, so slighly over 7g mem available for cache. The TestTable is snappy 
compressed, written with PE --size=100. Shell {{status 'simple'}} reports 
{{compressionRatio=0.2437}}. RandomRead test is run with --size=25, so with 
feature enabled, we should be at ~6g of compressed blocks.
h3. client perceived perf
With flag=false, I see slightly higher throughput (~7.5k vs ~8k) and slightly 
less gc (360ms vs 325ms).
h3. system load
CPU load looks slightly higher with flag=true (user time diff is negligible, 
system time more noticeable, no difference in iowait).
h3. blockcache utilization
There are definitely cache evictions happening with flag=false that do not 
happen with flag=true. The number of blocks cached is definitely higher with 
flag=true (~17.5k vs ~21.5k).
h3. IO
No difference in reported IO requests. Probably it's the OS buffer cache to the 
rescue here.

> [blockcache] lazy block decompression
> -
>
> Key: HBASE-11331
> URL: https://issues.apache.org/jira/browse/HBASE-11331
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Reporter: Nick Dimiduk
>Assignee: Nick Dimiduk
> Attachments: HBASE-11331.00.patch, HBASE-11331.01.patch, 
> HBASE-11331.02.patch, HBASE-11331LazyBlockDecompressperfcompare.pdf, 
> lazy-decompress.02.0.pdf
>
>
> Maintaining data in its compressed form in the block cache will greatly 
> increase our effective blockcache size and should show a meaning improvement 
> in cache hit rates in well designed applications. The idea here is to lazily 
> decompress/decrypt blocks when they're consumed, rather than as soon as 
> they're pulled off of disk.
> This is related to but less invasive than HBASE-8894.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Alex Newman (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11665?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alex Newman updated HBASE-11665:


   Resolution: Duplicate
Fix Version/s: (was: 0.99.0)
   Status: Resolved  (was: Patch Available)

DUP of HBASE-11611

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Jonathan Hsieh (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jonathan Hsieh updated HBASE-11535:
---

   Resolution: Fixed
Fix Version/s: 2.0.0
   0.99.0
 Hadoop Flags: Reviewed
   Status: Resolved  (was: Patch Available)

+1. 

Thanks for the patch virag!  The patches had essentially 2 +1's and shouldn't 
affect cause problems so I've committed to 0.98, branch-1 and master.

> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11546) Backport HBASE-11059 to 0.98

2014-08-06 Thread Andrew Purtell (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11546?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Purtell updated HBASE-11546:
---

Fix Version/s: 0.98.6
 Assignee: Virag Kothari

Moving in to 0.98.6. Thanks Virag. 

> Backport HBASE-11059 to 0.98
> 
>
> Key: HBASE-11546
> URL: https://issues.apache.org/jira/browse/HBASE-11546
> Project: HBase
>  Issue Type: Task
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.98.6
>
>
> Discuss concerns about backporting HBASE-11059 to 0.98



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Resolved] (HBASE-11667) Comment ClientScanner logic for NSREs.

2014-08-06 Thread Lars Hofhansl (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11667?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lars Hofhansl resolved HBASE-11667.
---

Resolution: Fixed

Committed doc update to all branches.

> Comment ClientScanner logic for NSREs.
> --
>
> Key: HBASE-11667
> URL: https://issues.apache.org/jira/browse/HBASE-11667
> Project: HBase
>  Issue Type: Improvement
>Reporter: Lars Hofhansl
>Assignee: Lars Hofhansl
>Priority: Minor
> Fix For: 0.99.0, 2.0.0, 0.94.23, 0.98.6
>
> Attachments: 11667-0.94.txt, 11667-doc-0.94.txt, 11667-trunk.txt, 
> HBASE-11667-0.98.patch, IntegrationTestBigLinkedListWithRegionMovement.patch
>
>
> We ran into an issue with Phoenix where a RegionObserver coprocessor 
> intercepts a scan and returns an aggregate (in this case a count) with a fake 
> row key. It turns out this does not work when the {{ClientScanner}} 
> encounters NSREs, as it uses the last key it saw to reset the scanner to try 
> again (which in this case would be the fake key).
> While this is arguably a rare case and one could also argue that a region 
> observer just shouldn't do this... While looking at {{ClientScanner}}'s code 
> I found this logic not necessary.
> A NSRE occurred because we contacted a region server with a key that it no 
> longer hosts. This is the start key, so it is always correct to retry with 
> this same key. That simplifies the ClientScanner logic and also make this 
> sort of coprocessors possible,



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread Jonathan Hsieh (JIRA)

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

Jonathan Hsieh commented on HBASE-11658:


This is nice.  I tested on trunk, currently testing on branch-1 and 0.98. 

[~enis], [~apurtell] do you guys want this in your respective branches?

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11658:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12660191/HBASE-11658.patch
  against trunk revision .
  ATTACHMENT ID: 12660191

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

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

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

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

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

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 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 site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.procedure.TestProcedureManager
  org.apache.hadoop.hbase.ipc.TestIPC
  org.apache.hadoop.hbase.master.TestClockSkewDetection

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

This message is automatically generated.

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Created] (HBASE-11686) Shell code should create a binding / irb workspace instead of polluting the root namespace

2014-08-06 Thread Sean Busbey (JIRA)
Sean Busbey created HBASE-11686:
---

 Summary: Shell code should create a binding / irb workspace 
instead of polluting the root namespace
 Key: HBASE-11686
 URL: https://issues.apache.org/jira/browse/HBASE-11686
 Project: HBase
  Issue Type: Improvement
Reporter: Sean Busbey
Priority: Minor


Right now, the shell builds a list of commands and then injects them into the 
root exectution's context

bin/hirb.rb
{code}
# Add commands to this namespace
@shell.export_commands(self)
{code}

hbase-shell/src/main/ruby/shell.rb
{code}
def export_commands(where)
  ::Shell.commands.keys.each do |cmd|
# here where is the IRB namespace
# this method just adds the call to the specified command
# which just references back to 'this' shell object
# a decently extensible way to add commands
where.send :instance_eval, <<-EOF
  def #{cmd}(*args)
ret = @shell.command('#{cmd}', *args)
puts
return ret
  end
EOF
  end
end
{code}

This is an unclean abstraction. For one, it requires that there be an instance 
variable in the main namespace called '@shell' without making that clear in the 
docs. Additionally, it complicates maintenance by breaking isolation.

We should update things so that shell can provide a binding for eval or a 
workspace for IRB execution and then use it directly when we construct our IRB 
session.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11686) Shell code should create a binding / irb workspace instead of polluting the root namespace

2014-08-06 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-11686:
-

an example of the complication that's just fall out of the main namespace 
pollution: you can't write tests that work on the exported command unless you 
know to construct an object that has a member variable '@shell'

> Shell code should create a binding / irb workspace instead of polluting the 
> root namespace
> --
>
> Key: HBASE-11686
> URL: https://issues.apache.org/jira/browse/HBASE-11686
> Project: HBase
>  Issue Type: Improvement
>Reporter: Sean Busbey
>Priority: Minor
>
> Right now, the shell builds a list of commands and then injects them into the 
> root exectution's context
> bin/hirb.rb
> {code}
> # Add commands to this namespace
> @shell.export_commands(self)
> {code}
> hbase-shell/src/main/ruby/shell.rb
> {code}
> def export_commands(where)
>   ::Shell.commands.keys.each do |cmd|
> # here where is the IRB namespace
> # this method just adds the call to the specified command
> # which just references back to 'this' shell object
> # a decently extensible way to add commands
> where.send :instance_eval, <<-EOF
>   def #{cmd}(*args)
> ret = @shell.command('#{cmd}', *args)
> puts
> return ret
>   end
> EOF
>   end
> end
> {code}
> This is an unclean abstraction. For one, it requires that there be an 
> instance variable in the main namespace called '@shell' without making that 
> clear in the docs. Additionally, it complicates maintenance by breaking 
> isolation.
> We should update things so that shell can provide a binding for eval or a 
> workspace for IRB execution and then use it directly when we construct our 
> IRB session.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Created] (HBASE-11687) Cancel postOpenDeployTasks if region opening is cancelled

2014-08-06 Thread Jimmy Xiang (JIRA)
Jimmy Xiang created HBASE-11687:
---

 Summary: Cancel postOpenDeployTasks if region opening is cancelled
 Key: HBASE-11687
 URL: https://issues.apache.org/jira/browse/HBASE-11687
 Project: HBase
  Issue Type: Bug
  Components: Region Assignment
Reporter: Jimmy Xiang
Assignee: Jimmy Xiang
 Fix For: 1.0.0


With ZK-less region assignment, if region opening is in postOpenDeployTasks 
step and the opening is cancelled, the region server will abort because it 
can't report the transition to the master. We should cancel postOpenDeployTasks 
instead. At least, there is no need to abort.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Alex Newman (JIRA)

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

Alex Newman commented on HBASE-11665:
-

Oh I checked to make sure we don't need this in branch-1 BTW. It passes on my 
laptop!

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread Sean Busbey (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11658?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Busbey updated HBASE-11658:


Attachment: HBASE_11658-v2.patch

Attaching v2 from the RB.

I took a look at the QA results. Since this patch only touches the shell ruby 
code, I don't see how it could have caused the test failures in hbase-server or 
the change in findbugs warnings.

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> HBASE_11658-v2.patch, test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11567) Write bulk load events to WAL

2014-08-06 Thread Alex Newman (JIRA)

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

Alex Newman commented on HBASE-11567:
-

I think we should wait to commit this until the reader side is done. I think 
figuring out those tests are important for the acceptability of this patch. 
That being said, I hope this is a useful first step for whoever (most likely 
me) has to do it.

> Write bulk load events to WAL
> -
>
> Key: HBASE-11567
> URL: https://issues.apache.org/jira/browse/HBASE-11567
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Enis Soztutar
>Assignee: Alex Newman
> Attachments: HBASE-11567-v1.patch, HBASE-11567-v2.patch
>
>
> Similar to writing flush (HBASE-11511), compaction(HBASE-2231) to WAL and 
> region open/close (HBASE-11512) , we should persist bulk load events to WAL.
> This is especially important for secondary region replicas, since we can use 
> this information to pick up primary regions' files from secondary replicas.
> A design doc for secondary replica replication can be found at HBASE-11183.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11686) Shell code should create a binding / irb workspace instead of polluting the root namespace

2014-08-06 Thread Sean Busbey (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11686?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Busbey updated HBASE-11686:


Component/s: shell

> Shell code should create a binding / irb workspace instead of polluting the 
> root namespace
> --
>
> Key: HBASE-11686
> URL: https://issues.apache.org/jira/browse/HBASE-11686
> Project: HBase
>  Issue Type: Improvement
>  Components: shell
>Reporter: Sean Busbey
>Priority: Minor
>
> Right now, the shell builds a list of commands and then injects them into the 
> root exectution's context
> bin/hirb.rb
> {code}
> # Add commands to this namespace
> @shell.export_commands(self)
> {code}
> hbase-shell/src/main/ruby/shell.rb
> {code}
> def export_commands(where)
>   ::Shell.commands.keys.each do |cmd|
> # here where is the IRB namespace
> # this method just adds the call to the specified command
> # which just references back to 'this' shell object
> # a decently extensible way to add commands
> where.send :instance_eval, <<-EOF
>   def #{cmd}(*args)
> ret = @shell.command('#{cmd}', *args)
> puts
> return ret
>   end
> EOF
>   end
> end
> {code}
> This is an unclean abstraction. For one, it requires that there be an 
> instance variable in the main namespace called '@shell' without making that 
> clear in the docs. Additionally, it complicates maintenance by breaking 
> isolation.
> We should update things so that shell can provide a binding for eval or a 
> workspace for IRB execution and then use it directly when we construct our 
> IRB session.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11678) BucketCache ramCache fills heap after running a few hours

2014-08-06 Thread stack (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11678?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-11678:
--

Attachment: 11678v2.txt

Managed to get a test in there for each of the modes that can happen adding 
item to BucketCache.  Lets see what the findbugs issues are.  Test failures 
look unrelated.

> BucketCache ramCache fills heap after running a few hours
> -
>
> Key: HBASE-11678
> URL: https://issues.apache.org/jira/browse/HBASE-11678
> Project: HBase
>  Issue Type: Bug
>  Components: BlockCache
>Affects Versions: 0.99.0, 0.98.5, 2.0.0
>Reporter: stack
>Assignee: stack
>Priority: Critical
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: 
> 0001-When-we-failed-add-an-entry-failing-with-a-CacheFull.patch, 11678v2.txt, 
> gc_over_12_hours_unevenblocks_with_lots_of_evictions.png
>
>
> Testing BucketCache, my heap filled after running for hours. Dumping heap, 
> culprit is the ramCache Map in BucketCache.  Tried running with more writer 
> threads but made no difference.  Trying to figure now how our accounting is 
> going wonky.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11603) Apply version of HADOOP-8027 to our JMXJsonServlet

2014-08-06 Thread stack (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11603?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-11603:
--

Attachment: 0001-HBASE-11603-Apply-version-of-HADOOP-8027-to-our-JMXJ.patch

Thanks [~ndimiduk]

Testing patch shows need to be more aggressive still.

> Apply version of HADOOP-8027 to our JMXJsonServlet
> --
>
> Key: HBASE-11603
> URL: https://issues.apache.org/jira/browse/HBASE-11603
> Project: HBase
>  Issue Type: Bug
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0
>
> Attachments: 
> 0001-HBASE-11603-Apply-version-of-HADOOP-8027-to-our-JMXJ.patch, 
> 0001-HBASE-11603-Apply-version-of-HADOOP-8027-to-our-JMXJ.patch, 11603.txt, 
> 11603v2.txt
>
>
> Running master, I see below when I poll /jmx servlet:
> {code}
> at javax.servlet.http.HttpServlet.service(HttpServlet.java:820)
>   at 
> org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:511)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1221)
>   at 
> org.apache.hadoop.hbase.http.lib.StaticUserWebFilter$StaticUserFilter.doFilter(StaticUserWebFilter.java:109)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.apache.hadoop.hbase.http.HttpServer$QuotingInputFilter.doFilter(HttpServer.java:1345)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.apache.hadoop.hbase.http.NoCacheFilter.doFilter(NoCacheFilter.java:45)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.apache.hadoop.hbase.http.NoCacheFilter.doFilter(NoCacheFilter.java:45)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:399)
>   at 
> org.mortbay.jetty.security.SecurityHandler.handle(SecurityHandler.java:216)
>   at 
> org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:182)
>   at 
> org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:766)
>   at org.mortbay.jetty.webapp.WebAppContext.handle(WebAppContext.java:450)
>   at 
> org.mortbay.jetty.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:230)
>   at 
> org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:152)
>   at org.mortbay.jetty.Server.handle(Server.java:326)
>   at 
> org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:542)
>   at 
> org.mortbay.jetty.HttpConnection$RequestHandler.headerComplete(HttpConnection.java:928)
>   at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:549)
>   at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:212)
>   at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:404)
>   at 
> org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:410)
>   at 
> org.mortbay.thread.QueuedThreadPool$PoolThread.run(QueuedThreadPool.java:582)
> Caused by: java.lang.UnsupportedOperationException: CollectionUsage threshold 
> is not supported
>   at 
> sun.management.MemoryPoolImpl.isCollectionUsageThresholdExceeded(MemoryPoolImpl.java:242)
>   at sun.reflect.GeneratedMethodAccessor67.invoke(Unknown Source)
>   at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>   at java.lang.reflect.Method.invoke(Method.java:606)
>   at sun.reflect.misc.Trampoline.invoke(MethodUtil.java:75)
>   at sun.reflect.GeneratedMethodAccessor46.invoke(Unknown Source)
>   at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>   at java.lang.reflect.Method.invoke(Method.java:606)
>   at sun.reflect.misc.MethodUtil.invoke(MethodUtil.java:279)
>   at 
> com.sun.jmx.mbeanserver.ConvertingMethod.invokeWithOpenReturn(ConvertingMethod.java:193)
>   at 
> com.sun.jmx.mbeanserver.ConvertingMethod.invokeWithOpenReturn(ConvertingMethod.java:175)
>   at 
> com.sun.jmx.mbeanserver.MXBeanIntrospector.invokeM2(MXBeanIntrospector.java:117)
>   at 
> com.sun.jmx.mbeanserver.MXBeanIntrospector.invokeM2(MXBeanIntrospector.java:54)
>   at 
> com.sun.jmx.mbeanserver.MBeanIntrospector.invokeM(MBeanIntrospector.java:237)
>   at 
> com.sun.jmx.mbeanserver.PerInterface.getAttribute(PerInterface.java:83)
>   at 
> com.sun.jmx.mbeanserver.MBeanSupport.getAttribute(MBeanSupport.java:206)
>   at javax.management.StandardMBean.getAttribute(StandardMBean.java:372)
>   at 
> com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.getAttribute(DefaultMBeanServerInterceptor.java:647)
> {code}
> This has been seen before 

[jira] [Updated] (HBASE-11603) Apply version of HADOOP-8027 to our JMXJsonServlet

2014-08-06 Thread stack (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11603?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-11603:
--

Status: Patch Available  (was: Reopened)

> Apply version of HADOOP-8027 to our JMXJsonServlet
> --
>
> Key: HBASE-11603
> URL: https://issues.apache.org/jira/browse/HBASE-11603
> Project: HBase
>  Issue Type: Bug
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0
>
> Attachments: 
> 0001-HBASE-11603-Apply-version-of-HADOOP-8027-to-our-JMXJ.patch, 
> 0001-HBASE-11603-Apply-version-of-HADOOP-8027-to-our-JMXJ.patch, 11603.txt, 
> 11603v2.txt
>
>
> Running master, I see below when I poll /jmx servlet:
> {code}
> at javax.servlet.http.HttpServlet.service(HttpServlet.java:820)
>   at 
> org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:511)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1221)
>   at 
> org.apache.hadoop.hbase.http.lib.StaticUserWebFilter$StaticUserFilter.doFilter(StaticUserWebFilter.java:109)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.apache.hadoop.hbase.http.HttpServer$QuotingInputFilter.doFilter(HttpServer.java:1345)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.apache.hadoop.hbase.http.NoCacheFilter.doFilter(NoCacheFilter.java:45)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.apache.hadoop.hbase.http.NoCacheFilter.doFilter(NoCacheFilter.java:45)
>   at 
> org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
>   at 
> org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:399)
>   at 
> org.mortbay.jetty.security.SecurityHandler.handle(SecurityHandler.java:216)
>   at 
> org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:182)
>   at 
> org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:766)
>   at org.mortbay.jetty.webapp.WebAppContext.handle(WebAppContext.java:450)
>   at 
> org.mortbay.jetty.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:230)
>   at 
> org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:152)
>   at org.mortbay.jetty.Server.handle(Server.java:326)
>   at 
> org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:542)
>   at 
> org.mortbay.jetty.HttpConnection$RequestHandler.headerComplete(HttpConnection.java:928)
>   at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:549)
>   at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:212)
>   at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:404)
>   at 
> org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:410)
>   at 
> org.mortbay.thread.QueuedThreadPool$PoolThread.run(QueuedThreadPool.java:582)
> Caused by: java.lang.UnsupportedOperationException: CollectionUsage threshold 
> is not supported
>   at 
> sun.management.MemoryPoolImpl.isCollectionUsageThresholdExceeded(MemoryPoolImpl.java:242)
>   at sun.reflect.GeneratedMethodAccessor67.invoke(Unknown Source)
>   at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>   at java.lang.reflect.Method.invoke(Method.java:606)
>   at sun.reflect.misc.Trampoline.invoke(MethodUtil.java:75)
>   at sun.reflect.GeneratedMethodAccessor46.invoke(Unknown Source)
>   at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>   at java.lang.reflect.Method.invoke(Method.java:606)
>   at sun.reflect.misc.MethodUtil.invoke(MethodUtil.java:279)
>   at 
> com.sun.jmx.mbeanserver.ConvertingMethod.invokeWithOpenReturn(ConvertingMethod.java:193)
>   at 
> com.sun.jmx.mbeanserver.ConvertingMethod.invokeWithOpenReturn(ConvertingMethod.java:175)
>   at 
> com.sun.jmx.mbeanserver.MXBeanIntrospector.invokeM2(MXBeanIntrospector.java:117)
>   at 
> com.sun.jmx.mbeanserver.MXBeanIntrospector.invokeM2(MXBeanIntrospector.java:54)
>   at 
> com.sun.jmx.mbeanserver.MBeanIntrospector.invokeM(MBeanIntrospector.java:237)
>   at 
> com.sun.jmx.mbeanserver.PerInterface.getAttribute(PerInterface.java:83)
>   at 
> com.sun.jmx.mbeanserver.MBeanSupport.getAttribute(MBeanSupport.java:206)
>   at javax.management.StandardMBean.getAttribute(StandardMBean.java:372)
>   at 
> com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.getAttribute(DefaultMBeanServerInterceptor.java:647)
> {code}
> This has been seen before over in HADOOP-8027.  We get it since we brought 
> jetty server local from hadoop.  I don't think it in branc

[jira] [Updated] (HBASE-11331) [blockcache] lazy block decompression

2014-08-06 Thread Nick Dimiduk (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11331?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nick Dimiduk updated HBASE-11331:
-

Attachment: lazy-decompress.02.1.pdf

re: lazy-decompress.02.1.pdf

Attaching some metrics after running v02 with less os page cache and feature 
enabled/disabled.

This time the total RS heap is increased to 20g, so the effective blockcache 
size is ~12g. The machine has 24g of RAM, so this squeezes the OS-available 
memory down to ~4g -- when the blockcache is full. I've added blockcache size 
and OS page cache to the charts  Same table, same compression ratio, same PE 
randomRead test, this time run with --size=45, so I expect ~11g of compressed 
blocks.

I expected this to be an IO-bound workload with flag=false but to be 
cache-at-capacity for flag=true. The massive evictions look to me like the 
blockcache heapsize calculation is based on the decompressed size, not 
compressed size. Updating patch.

> [blockcache] lazy block decompression
> -
>
> Key: HBASE-11331
> URL: https://issues.apache.org/jira/browse/HBASE-11331
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Reporter: Nick Dimiduk
>Assignee: Nick Dimiduk
> Attachments: HBASE-11331.00.patch, HBASE-11331.01.patch, 
> HBASE-11331.02.patch, HBASE-11331LazyBlockDecompressperfcompare.pdf, 
> lazy-decompress.02.0.pdf, lazy-decompress.02.1.pdf
>
>
> Maintaining data in its compressed form in the block cache will greatly 
> increase our effective blockcache size and should show a meaning improvement 
> in cache hit rates in well designed applications. The idea here is to lazily 
> decompress/decrypt blocks when they're consumed, rather than as soon as 
> they're pulled off of disk.
> This is related to but less invasive than HBASE-8894.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11331) [blockcache] lazy block decompression

2014-08-06 Thread stack (JIRA)

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

stack commented on HBASE-11331:
---

Any chance of our fixing compressor reuse. As you say, I'd imagine it'd mess up 
any possibility of nice  numbers when this feature enabled.  I'm game to rerun 
test when you say its ready.  That a fancy graph tool on front a tsdb? 
[~ndimiduk]?  Nice graphs.

> [blockcache] lazy block decompression
> -
>
> Key: HBASE-11331
> URL: https://issues.apache.org/jira/browse/HBASE-11331
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Reporter: Nick Dimiduk
>Assignee: Nick Dimiduk
> Attachments: HBASE-11331.00.patch, HBASE-11331.01.patch, 
> HBASE-11331.02.patch, HBASE-11331LazyBlockDecompressperfcompare.pdf, 
> lazy-decompress.02.0.pdf, lazy-decompress.02.1.pdf
>
>
> Maintaining data in its compressed form in the block cache will greatly 
> increase our effective blockcache size and should show a meaning improvement 
> in cache hit rates in well designed applications. The idea here is to lazily 
> decompress/decrypt blocks when they're consumed, rather than as soon as 
> they're pulled off of disk.
> This is related to but less invasive than HBASE-8894.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Created] (HBASE-11688) Add histograms of key attributes to server front pages

2014-08-06 Thread stack (JIRA)
stack created HBASE-11688:
-

 Summary: Add histograms of key attributes to server front pages
 Key: HBASE-11688
 URL: https://issues.apache.org/jira/browse/HBASE-11688
 Project: HBase
  Issue Type: Improvement
Reporter: stack
Priority: Minor


Messing with histograms of late, seems a pity we don't show in UI histograms of 
basic recent events and metrics. In past we've tried to keep history in a 
history table and we've talked of bringing this back as data source for being 
able to mine what has been happening of late and in the recent past but 
meantime, we could keep running in-memory histograms of recent history and show 
cute little histo-diagrams of hits, compactions, flushes... Would be nice 
little project with a high profile.  It'd not be a replacement for cluster 
monitoring but more just an at-a-glance high-level dashboard that gives a clue 
on "...what has my cluster been up to of late?"



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11688) Add histograms of key attributes to server front pages

2014-08-06 Thread stack (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11688?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stack updated HBASE-11688:
--

Component/s: monitoring

> Add histograms of key attributes to server front pages
> --
>
> Key: HBASE-11688
> URL: https://issues.apache.org/jira/browse/HBASE-11688
> Project: HBase
>  Issue Type: Improvement
>  Components: monitoring
>Reporter: stack
>Priority: Minor
>  Labels: beginner
>
> Messing with histograms of late, seems a pity we don't show in UI histograms 
> of basic recent events and metrics. In past we've tried to keep history in a 
> history table and we've talked of bringing this back as data source for being 
> able to mine what has been happening of late and in the recent past but 
> meantime, we could keep running in-memory histograms of recent history and 
> show cute little histo-diagrams of hits, compactions, flushes... Would be 
> nice little project with a high profile.  It'd not be a replacement for 
> cluster monitoring but more just an at-a-glance high-level dashboard that 
> gives a clue on "...what has my cluster been up to of late?"



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11567) Write bulk load events to WAL

2014-08-06 Thread stack (JIRA)

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

stack commented on HBASE-11567:
---

Took a quick look.  Looks good to me.  We could commit this but agree probably 
better if read-side goes in around same time.

> Write bulk load events to WAL
> -
>
> Key: HBASE-11567
> URL: https://issues.apache.org/jira/browse/HBASE-11567
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Enis Soztutar
>Assignee: Alex Newman
> Attachments: HBASE-11567-v1.patch, HBASE-11567-v2.patch
>
>
> Similar to writing flush (HBASE-11511), compaction(HBASE-2231) to WAL and 
> region open/close (HBASE-11512) , we should persist bulk load events to WAL.
> This is especially important for secondary region replicas, since we can use 
> this information to pick up primary regions' files from secondary replicas.
> A design doc for secondary replica replication can be found at HBASE-11183.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11567) Write bulk load events to WAL

2014-08-06 Thread Alex Newman (JIRA)

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

Alex Newman commented on HBASE-11567:
-

Daww thanks stack. One question I have is, currently I am logging the request 
by the user for HBase HFile bulk loading. That being said, does it make sense 
to have a different WAL entry when it completes successfully?

> Write bulk load events to WAL
> -
>
> Key: HBASE-11567
> URL: https://issues.apache.org/jira/browse/HBASE-11567
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Enis Soztutar
>Assignee: Alex Newman
> Attachments: HBASE-11567-v1.patch, HBASE-11567-v2.patch
>
>
> Similar to writing flush (HBASE-11511), compaction(HBASE-2231) to WAL and 
> region open/close (HBASE-11512) , we should persist bulk load events to WAL.
> This is especially important for secondary region replicas, since we can use 
> this information to pick up primary regions' files from secondary replicas.
> A design doc for secondary replica replication can be found at HBASE-11183.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread stack (JIRA)

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

stack commented on HBASE-11658:
---

Skimmed.  It looks great.

Failures are infrastructure issues.  See 
https://issues.apache.org/jira/browse/BUILDS-4

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> HBASE_11658-v2.patch, test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread stack (JIRA)

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

stack commented on HBASE-11611:
---

Skimmed.  [~jxiang] This looks to be candidate for best patch of 2014!  Its 
great.

Can you get hadoopqa to apply it?  I think we just apply if it passes.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11535:


SUCCESS: Integrated in HBase-0.98 #438 (See 
[https://builds.apache.org/job/HBase-0.98/438/])
HBASE-11535 ReplicationPeer map is not thread safe (Virag Kothari) (jon: rev 
c8812737bb53016ec18738061b9c2a18ca8c0ad8)
* 
hbase-client/src/main/java/org/apache/hadoop/hbase/replication/ReplicationPeersZKImpl.java


> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11331) [blockcache] lazy block decompression

2014-08-06 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-11331:
--

bq. Any chance of our fixing compressor reuse.

I'll look into it. If anything, it'll be a separate ticket.

bq. I'm game to rerun test when you say its ready.

Hold off on rerunning until I verify/refute my hypothesis on the blockcache 
heapsize stuff.

bq. Nice graphs

Awe shucks, thanks boss. This is Grafana running on top of OpenTSDB. No 
offense, but I find it nicer to use these dashboards than the gnuplot stuff we 
get otherwise. I'm still learning it, but I'll attach the dashboard json file I 
used to make the latest figures.

> [blockcache] lazy block decompression
> -
>
> Key: HBASE-11331
> URL: https://issues.apache.org/jira/browse/HBASE-11331
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Reporter: Nick Dimiduk
>Assignee: Nick Dimiduk
> Attachments: HBASE-11331.00.patch, HBASE-11331.01.patch, 
> HBASE-11331.02.patch, HBASE-11331LazyBlockDecompressperfcompare.pdf, 
> lazy-decompress.02.0.pdf, lazy-decompress.02.1.pdf
>
>
> Maintaining data in its compressed form in the block cache will greatly 
> increase our effective blockcache size and should show a meaning improvement 
> in cache hit rates in well designed applications. The idea here is to lazily 
> decompress/decrypt blocks when they're consumed, rather than as soon as 
> they're pulled off of disk.
> This is related to but less invasive than HBASE-8894.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread stack (JIRA)

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

stack commented on HBASE-11665:
---

bq. Oh I checked to make sure we don't need this in branch-1

Sorry [~posix4e] We need this patch or not?  Thanks.

[~jxiang] this flag is set true in branch-1 already?  We don't need set it in 
the test?



> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11331) [blockcache] lazy block decompression

2014-08-06 Thread Nick Dimiduk (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11331?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nick Dimiduk updated HBASE-11331:
-

Attachment: lazy-decompress.02.1.json

Attach lazy-decompress.02.1.json, used with grafana 1.7rc for the legend 
aliasing.

> [blockcache] lazy block decompression
> -
>
> Key: HBASE-11331
> URL: https://issues.apache.org/jira/browse/HBASE-11331
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Reporter: Nick Dimiduk
>Assignee: Nick Dimiduk
> Attachments: HBASE-11331.00.patch, HBASE-11331.01.patch, 
> HBASE-11331.02.patch, HBASE-11331LazyBlockDecompressperfcompare.pdf, 
> lazy-decompress.02.0.pdf, lazy-decompress.02.1.json, lazy-decompress.02.1.pdf
>
>
> Maintaining data in its compressed form in the block cache will greatly 
> increase our effective blockcache size and should show a meaning improvement 
> in cache hit rates in well designed applications. The idea here is to lazily 
> decompress/decrypt blocks when they're consumed, rather than as soon as 
> they're pulled off of disk.
> This is related to but less invasive than HBASE-8894.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Alex Newman (JIRA)

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

Alex Newman commented on HBASE-11665:
-

We will not need it in branch-1

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11665) TestRegionReplicas doesn't work for me

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11665:
-

The flag is set to true by default in branch-1. So we don't need set it in the 
test.

> TestRegionReplicas doesn't work for me
> --
>
> Key: HBASE-11665
> URL: https://issues.apache.org/jira/browse/HBASE-11665
> Project: HBase
>  Issue Type: Bug
>Reporter: Alex Newman
>Assignee: Alex Newman
> Fix For: 2.0.0
>
> Attachments: HBASE-11665-v1.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11660) Make WAL reader follow contract for java.io.InputStream.available()

2014-08-06 Thread Andrew Purtell (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Purtell updated HBASE-11660:
---

Affects Version/s: 0.98.6
   2.0.0
   0.99.0

> Make WAL reader follow contract for java.io.InputStream.available()
> ---
>
> Key: HBASE-11660
> URL: https://issues.apache.org/jira/browse/HBASE-11660
> Project: HBase
>  Issue Type: Bug
>  Components: wal
>Affects Versions: 0.99.0, 2.0.0, 0.98.6
>Reporter: Eric Hanson
>Priority: Minor
> Attachments: hbase-11660.01.patch
>
>
> In the process of building support to running HBase on Microsoft Azure 
> HDInsight, I hit an issue in the HBase WAL reading process that took a lot of 
> time to debug. The WAL reading code depends on available() for the log 
> InputStream never returing 0 until end of file. This is not the same as the 
> contract in java.io.InputStream for available. 
> To prevent future grief for others that may want to port HBase onto storage 
> systems other than HDFS, I propose to change the HBase WAL reader so it does 
> not assume that EOF has been reached when available() == 0. It instead would 
> treat available only as described in InputStream, i.e. available() is merely 
> the number of bytes that could be read from the stream without blocking. That 
> could be 0 even before EOF.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11685) Incr/decr on the reference count of HConnectionImplementation need be atomic

2014-08-06 Thread Andrew Purtell (JIRA)

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

Andrew Purtell commented on HBASE-11685:


This change does not preserve the old behavior where we avoid taking refCount 
negative. A concern? We should warn loudly if this goes negative at least I 
think.

{code}
 /**
  * Decrement this client's reference count.
  */
 void decCount() {
-  if (refCount > 0) {
---refCount;
-  }
+  refCount.decrementAndGet();
 }
{code}


> Incr/decr on the reference count of HConnectionImplementation need be atomic 
> -
>
> Key: HBASE-11685
> URL: https://issues.apache.org/jira/browse/HBASE-11685
> Project: HBase
>  Issue Type: Bug
>  Components: Client
>Reporter: Liu Shaohui
>Assignee: Liu Shaohui
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-11685-trunk-v1.diff, HBASE-11685-trunk-v2.diff
>
>
> Currently, the incr/decr operation on the ref count of 
> HConnectionImplementation are not atomic. This may cause that the ref count 
> always be larger than 0 and  the connection never be closed.
> {code}
> /**
>  * Increment this client's reference count.
>  */
> void incCount() {
>   ++refCount;
> }
> /**
>  * Decrement this client's reference count.
>  */
> void decCount() {
>   if (refCount > 0) {
> --refCount;
>   }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11546) Backport HBASE-11059 (ZK-less region assignment) to 0.98

2014-08-06 Thread Andrew Purtell (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11546?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Purtell updated HBASE-11546:
---

Summary: Backport HBASE-11059 (ZK-less region assignment) to 0.98  (was: 
Backport HBASE-11059 to 0.98)

> Backport HBASE-11059 (ZK-less region assignment) to 0.98
> 
>
> Key: HBASE-11546
> URL: https://issues.apache.org/jira/browse/HBASE-11546
> Project: HBase
>  Issue Type: Task
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.98.6
>
>
> Discuss concerns about backporting HBASE-11059 to 0.98



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11678) BucketCache ramCache fills heap after running a few hours

2014-08-06 Thread Nick Dimiduk (JIRA)

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

Nick Dimiduk commented on HBASE-11678:
--

patch v2:

{noformat}
+"; adjust BucketCache sizes " + CacheConfig.BUCKET_CACHE_BUCKETS_KEY +
+" to accomodate if size seems reasonable and you want it cached.");
{noformat}

Nice, I like it.

{noformat}
+private volatile boolean writerEnabled = true;
{noformat}

Why mark this volatile?

{noformat}
-  if (cacheEnabled
-  && (now - ioErrorStartTime) > this.ioErrorsTolerationDuration) {
-LOG.error("IO errors duration time has exceeded "
-+ ioErrorsTolerationDuration
-+ "ms, disabing cache, please check your IOEngine");
+  if (cacheEnabled && (now - ioErrorStartTime) > 
this.ioErrorsTolerationDuration) {
+LOG.error("IO errors duration time has exceeded " + 
ioErrorsTolerationDuration +
+  "ms, disabing cache, please check your IOEngine");
 disableCache();
{noformat}

I know there's no new functionality here, but shouldn't this be an RS-abort 
kind of offense?

{noformat}
+this.bc = new BucketCache("heap", capacity, 1, new int [] {1}, 
writerThreadsCount,
{noformat}

nit: isn't "heap" and friends in an Enum somewhere? ... Oh, there's not. 
Probably should be.

Nice new tests. Are you able to verify that testTooBigEntry fails on current 
master? Maybe too much refactoring is necessary to confirm.

+1

> BucketCache ramCache fills heap after running a few hours
> -
>
> Key: HBASE-11678
> URL: https://issues.apache.org/jira/browse/HBASE-11678
> Project: HBase
>  Issue Type: Bug
>  Components: BlockCache
>Affects Versions: 0.99.0, 0.98.5, 2.0.0
>Reporter: stack
>Assignee: stack
>Priority: Critical
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: 
> 0001-When-we-failed-add-an-entry-failing-with-a-CacheFull.patch, 11678v2.txt, 
> gc_over_12_hours_unevenblocks_with_lots_of_evictions.png
>
>
> Testing BucketCache, my heap filled after running for hours. Dumping heap, 
> culprit is the ramCache Map in BucketCache.  Tried running with more writer 
> threads but made no difference.  Trying to figure now how our accounting is 
> going wonky.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11667) Comment ClientScanner logic for NSREs.

2014-08-06 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11667:


FAILURE: Integrated in HBase-0.94-JDK7 #167 (See 
[https://builds.apache.org/job/HBase-0.94-JDK7/167/])
HBASE-11667 Comment ClientScanner logic for NSREs. (larsh: rev 
c51e19afec1925de7769c24e0dbd2bfcf90a184e)
* src/main/java/org/apache/hadoop/hbase/client/ClientScanner.java


> Comment ClientScanner logic for NSREs.
> --
>
> Key: HBASE-11667
> URL: https://issues.apache.org/jira/browse/HBASE-11667
> Project: HBase
>  Issue Type: Improvement
>Reporter: Lars Hofhansl
>Assignee: Lars Hofhansl
>Priority: Minor
> Fix For: 0.99.0, 2.0.0, 0.94.23, 0.98.6
>
> Attachments: 11667-0.94.txt, 11667-doc-0.94.txt, 11667-trunk.txt, 
> HBASE-11667-0.98.patch, IntegrationTestBigLinkedListWithRegionMovement.patch
>
>
> We ran into an issue with Phoenix where a RegionObserver coprocessor 
> intercepts a scan and returns an aggregate (in this case a count) with a fake 
> row key. It turns out this does not work when the {{ClientScanner}} 
> encounters NSREs, as it uses the last key it saw to reset the scanner to try 
> again (which in this case would be the fake key).
> While this is arguably a rare case and one could also argue that a region 
> observer just shouldn't do this... While looking at {{ClientScanner}}'s code 
> I found this logic not necessary.
> A NSRE occurred because we contacted a region server with a key that it no 
> longer hosts. This is the start key, so it is always correct to retry with 
> this same key. That simplifies the ClientScanner logic and also make this 
> sort of coprocessors possible,



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11658) Piped commands to hbase shell should return non-zero if shell command failed.

2014-08-06 Thread Jonathan Hsieh (JIRA)

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

Jonathan Hsieh commented on HBASE-11658:


lgtm -- applies to branch-1 and 0.98 and looks like it works on both.  waiting 
for their feedback before I commit to master and those branches.

> Piped commands to hbase shell should return non-zero if shell command failed.
> -
>
> Key: HBASE-11658
> URL: https://issues.apache.org/jira/browse/HBASE-11658
> Project: HBase
>  Issue Type: Bug
>  Components: shell
>Affects Versions: 2.0.0
>Reporter: Jonathan Hsieh
>Assignee: Sean Busbey
> Attachments: HBASE-11658.manual_test.log, HBASE-11658.patch, 
> HBASE_11658-v2.patch, test_shell.sh
>
>
> See HBASE-11655.  We would like the shell to return non-zero when a command 
> that has been piped to it fails in "scripting mode".  This could be due to 
> invalid commands or commands issued with invalid arguments.
> This would lower the barrier to entry for hbase admins so they could 
> effectively script some operations.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11667) Comment ClientScanner logic for NSREs.

2014-08-06 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11667:


FAILURE: Integrated in HBase-TRUNK #5375 (See 
[https://builds.apache.org/job/HBase-TRUNK/5375/])
HBASE-11667 Comment ClientScanner logic for NSREs. (larsh: rev 
25e6baee424602942afbe40b01ea0b4cff1ea9b1)
* hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClientScanner.java


> Comment ClientScanner logic for NSREs.
> --
>
> Key: HBASE-11667
> URL: https://issues.apache.org/jira/browse/HBASE-11667
> Project: HBase
>  Issue Type: Improvement
>Reporter: Lars Hofhansl
>Assignee: Lars Hofhansl
>Priority: Minor
> Fix For: 0.99.0, 2.0.0, 0.94.23, 0.98.6
>
> Attachments: 11667-0.94.txt, 11667-doc-0.94.txt, 11667-trunk.txt, 
> HBASE-11667-0.98.patch, IntegrationTestBigLinkedListWithRegionMovement.patch
>
>
> We ran into an issue with Phoenix where a RegionObserver coprocessor 
> intercepts a scan and returns an aggregate (in this case a count) with a fake 
> row key. It turns out this does not work when the {{ClientScanner}} 
> encounters NSREs, as it uses the last key it saw to reset the scanner to try 
> again (which in this case would be the fake key).
> While this is arguably a rare case and one could also argue that a region 
> observer just shouldn't do this... While looking at {{ClientScanner}}'s code 
> I found this logic not necessary.
> A NSRE occurred because we contacted a region server with a key that it no 
> longer hosts. This is the start key, so it is always correct to retry with 
> this same key. That simplifies the ClientScanner logic and also make this 
> sort of coprocessors possible,



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11535) ReplicationPeer map is not thread safe

2014-08-06 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11535:


FAILURE: Integrated in HBase-TRUNK #5375 (See 
[https://builds.apache.org/job/HBase-TRUNK/5375/])
HBASE-11535 ReplicationPeer map is not thread safe (Virag Kothari) (jmhsieh: 
rev 6dee406bf359eec31883cdb2714399a8161a4c26)
* 
hbase-client/src/main/java/org/apache/hadoop/hbase/replication/ReplicationPeersZKImpl.java


> ReplicationPeer map is not thread safe
> --
>
> Key: HBASE-11535
> URL: https://issues.apache.org/jira/browse/HBASE-11535
> Project: HBase
>  Issue Type: Bug
>  Components: Replication
>Affects Versions: 0.98.4
>Reporter: Virag Kothari
>Assignee: Virag Kothari
> Fix For: 0.99.0, 2.0.0, 0.98.6
>
> Attachments: HBASE-11535.patch, HBASE-11535.patch, 
> HBASE-11535_98.patch
>
>
> ReplicationPeersZKImpl has a map of Peer id to ReplicationPeer which is 
> accessed by replication source threads through getStatusOfPeer(). So, it 
> needs to be thread safe.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11331) [blockcache] lazy block decompression

2014-08-06 Thread Andrew Purtell (JIRA)

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

Andrew Purtell commented on HBASE-11331:


bq. Grafana running on top of OpenTSDB

Very nice. Is there a way to autogen the JSON? Pardon for hijacking the issue.

> [blockcache] lazy block decompression
> -
>
> Key: HBASE-11331
> URL: https://issues.apache.org/jira/browse/HBASE-11331
> Project: HBase
>  Issue Type: Improvement
>  Components: regionserver
>Reporter: Nick Dimiduk
>Assignee: Nick Dimiduk
> Attachments: HBASE-11331.00.patch, HBASE-11331.01.patch, 
> HBASE-11331.02.patch, HBASE-11331LazyBlockDecompressperfcompare.pdf, 
> lazy-decompress.02.0.pdf, lazy-decompress.02.1.json, lazy-decompress.02.1.pdf
>
>
> Maintaining data in its compressed form in the block cache will greatly 
> increase our effective blockcache size and should show a meaning improvement 
> in cache hit rates in well designed applications. The idea here is to lazily 
> decompress/decrypt blocks when they're consumed, rather than as soon as 
> they're pulled off of disk.
> This is related to but less invasive than HBASE-8894.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11681) Update and move doc about disabling the WAL

2014-08-06 Thread Jonathan Hsieh (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jonathan Hsieh updated HBASE-11681:
---

Affects Version/s: 2.0.0
   0.99.0

> Update and move doc about disabling the WAL
> ---
>
> Key: HBASE-11681
> URL: https://issues.apache.org/jira/browse/HBASE-11681
> Project: HBase
>  Issue Type: Bug
>  Components: documentation
>Affects Versions: 0.99.0, 2.0.0
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11681.patch
>
>
> Move the docs about disabling the WAL from the Performance section to the WAL 
> section, point to the new location, and add info about getDurability and 
> setDurability methods. Leave the big fat warnings about data loss in both the 
> Performance and WAL sections.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11681) Update and move doc about disabling the WAL

2014-08-06 Thread Jonathan Hsieh (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jonathan Hsieh updated HBASE-11681:
---

  Resolution: Fixed
Hadoop Flags: Reviewed
  Status: Resolved  (was: Patch Available)

lgtm, committing to branch-1 and master. 

> Update and move doc about disabling the WAL
> ---
>
> Key: HBASE-11681
> URL: https://issues.apache.org/jira/browse/HBASE-11681
> Project: HBase
>  Issue Type: Bug
>  Components: documentation
>Affects Versions: 0.99.0, 2.0.0
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11681.patch
>
>
> Move the docs about disabling the WAL from the Performance section to the WAL 
> section, point to the new location, and add info about getDurability and 
> setDurability methods. Leave the big fat warnings about data loss in both the 
> Performance and WAL sections.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Updated] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jimmy Xiang updated HBASE-11611:


Attachment: hbase-11611_v2.patch

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch, 
> hbase-11611_v2.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Jimmy Xiang (JIRA)

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

Jimmy Xiang commented on HBASE-11611:
-

Attached v2. Hopefully it's ok with hadoopqa. Thanks.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch, 
> hbase-11611_v2.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Reopened] (HBASE-11673) TestIOFencing#testFencingAroundCompactionAfterWALSync fails

2014-08-06 Thread Ted Yu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-11673?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ted Yu reopened HBASE-11673:



The test fails in HBase-TRUNK build #5375 

Mind taking a look ?

> TestIOFencing#testFencingAroundCompactionAfterWALSync fails
> ---
>
> Key: HBASE-11673
> URL: https://issues.apache.org/jira/browse/HBASE-11673
> Project: HBase
>  Issue Type: Test
>Reporter: Qiang Tian
>Assignee: Sergey Soldatov
> Fix For: 2.0.0
>
> Attachments: HBASE_11673-v1.patch
>
>
> got several test failure on the latest build:
> {quote}
> [tianq@bdvm101 surefire-reports]$ ls -1t|grep "Tests run" * |grep "<<< 
> FAILURE" 
> org.apache.hadoop.hbase.client.TestReplicasClient.txt:Tests run: 1, Failures: 
> 0, Errors: 1, Skipped: 0, Time elapsed: 38.706 sec <<< FAILURE!
> org.apache.hadoop.hbase.master.TestMasterOperationsForRegionReplicas.txt:Tests
>  run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 30.669 sec <<< 
> FAILURE!
> org.apache.hadoop.hbase.regionserver.TestRegionReplicas.txt:Tests run: 1, 
> Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 39.113 sec <<< FAILURE!
> org.apache.hadoop.hbase.TestIOFencing.txt:Tests run: 2, Failures: 1, Errors: 
> 0, Skipped: 0, Time elapsed: 177.071 sec <<< FAILURE!
> {quote} 
> the first one:
> {quote} 
>  type="java.lang.AssertionError">java.lang.AssertionError: Timed out waiting 
> for the region to flush
> >-at org.junit.Assert.fail(Assert.java:88)
> >-at org.junit.Assert.assertTrue(Assert.java:41)
> >-at org.apache.hadoop.hbase.TestIOFencing.doTest(TestIOFencing.java:291)
> >-at 
> >org.apache.hadoop.hbase.TestIOFencing.testFencingAroundCompactionAfterWALSync(TestIOFencing.java:236)
> >-at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> >-at 
> >sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
> >-at 
> >sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> >-at java.lang.reflect.Method.invoke(Method.java:606)
> {quote}



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11611) Clean up ZK-based region assignment

2014-08-06 Thread Alex Newman (JIRA)

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

Alex Newman commented on HBASE-11611:
-

That much red makes me shed tears of joy.

> Clean up ZK-based region assignment
> ---
>
> Key: HBASE-11611
> URL: https://issues.apache.org/jira/browse/HBASE-11611
> Project: HBase
>  Issue Type: Improvement
>  Components: Region Assignment
>Reporter: Jimmy Xiang
>Assignee: Jimmy Xiang
> Fix For: 2.0.0
>
> Attachments: hbase-11611.patch, hbase-11611_v1.patch, 
> hbase-11611_v2.patch
>
>
> We can clean up the ZK-based region assignment code and use the ZK-less one 
> in the master branch, to make the code easier to understand and maintain.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11681) Update and move doc about disabling the WAL

2014-08-06 Thread Jonathan Hsieh (JIRA)

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

Jonathan Hsieh commented on HBASE-11681:


thanks misty!

> Update and move doc about disabling the WAL
> ---
>
> Key: HBASE-11681
> URL: https://issues.apache.org/jira/browse/HBASE-11681
> Project: HBase
>  Issue Type: Bug
>  Components: documentation
>Affects Versions: 0.99.0, 2.0.0
>Reporter: Misty Stanley-Jones
>Assignee: Misty Stanley-Jones
> Attachments: HBASE-11681.patch
>
>
> Move the docs about disabling the WAL from the Performance section to the WAL 
> section, point to the new location, and add info about getDurability and 
> setDurability methods. Leave the big fat warnings about data loss in both the 
> Performance and WAL sections.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-4593) Design and document the official procedure for posting patches, commits, commit messages, etc. to smooth process and make integration with tools easier

2014-08-06 Thread Misty Stanley-Jones (JIRA)

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

Misty Stanley-Jones commented on HBASE-4593:


>From an email from [~apurtell] 
{code}
-- Forwarded message --
From: Andrew Purtell 
Date: Fri, Aug 1, 2014 at 10:32 AM
Subject: Friendly reminder to use git cherry-pick and git rebase instead of git 
merge
To: "d...@hbase.apache.org" 


We previously agreed to avoid merge commits appearing in our git history.

Found a merge commit on 0.98 branch.

Thanks!
{code}

> Design and document the official procedure for posting patches, commits, 
> commit messages, etc. to smooth process and make integration with tools easier
> ---
>
> Key: HBASE-4593
> URL: https://issues.apache.org/jira/browse/HBASE-4593
> Project: HBase
>  Issue Type: Task
>  Components: documentation
>Reporter: Jonathan Gray
>Assignee: Misty Stanley-Jones
>
> I have been building a tool (currently called reposync) to help me keep the 
> internal FB hbase-92-based branch up-to-date with the public branches.
> Various inconsistencies in our process has made it difficult to automate a 
> lot of this stuff.
> I'd like to work with everyone to come up with the official best practices 
> and stick to it.
> I welcome all suggestions.  Among some of the things I'd like to nail down:
> - Commit message format
> - Best practice and commit message format for multiple commits
> - Multiple commits per jira vs. jira per commit, what are the exceptions and 
> when
> - Affects vs. Fix versions
> - Potential usage of [tags] in commit messages for things like book, scripts, 
> shell... maybe even whatever is in the components field?
> - Increased usage of JIRA tags or labels to mark exactly which repos a JIRA 
> has been committed to (potentially even internal repos?  ways for a tool to 
> keep track in JIRA?)
> We also need to be more strict about some things if we want to follow Apache 
> guidelines.  For example, all final versions of a patch must be attached to 
> JIRA so that the author properly assigns it to Apache.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-4593) Design and document the official procedure for posting patches, commits, commit messages, etc. to smooth process and make integration with tools easier

2014-08-06 Thread Misty Stanley-Jones (JIRA)

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

Misty Stanley-Jones commented on HBASE-4593:


What about ReviewBoard? Can we do better than 
http://hbase.apache.org/book.html#reviewboard? What is "larger patches"? Can 
someone describe the workflow of ReviewBoard for me? I have never used it.

> Design and document the official procedure for posting patches, commits, 
> commit messages, etc. to smooth process and make integration with tools easier
> ---
>
> Key: HBASE-4593
> URL: https://issues.apache.org/jira/browse/HBASE-4593
> Project: HBase
>  Issue Type: Task
>  Components: documentation
>Reporter: Jonathan Gray
>Assignee: Misty Stanley-Jones
>
> I have been building a tool (currently called reposync) to help me keep the 
> internal FB hbase-92-based branch up-to-date with the public branches.
> Various inconsistencies in our process has made it difficult to automate a 
> lot of this stuff.
> I'd like to work with everyone to come up with the official best practices 
> and stick to it.
> I welcome all suggestions.  Among some of the things I'd like to nail down:
> - Commit message format
> - Best practice and commit message format for multiple commits
> - Multiple commits per jira vs. jira per commit, what are the exceptions and 
> when
> - Affects vs. Fix versions
> - Potential usage of [tags] in commit messages for things like book, scripts, 
> shell... maybe even whatever is in the components field?
> - Increased usage of JIRA tags or labels to mark exactly which repos a JIRA 
> has been committed to (potentially even internal repos?  ways for a tool to 
> keep track in JIRA?)
> We also need to be more strict about some things if we want to follow Apache 
> guidelines.  For example, all final versions of a patch must be attached to 
> JIRA so that the author properly assigns it to Apache.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


  1   2   3   >