[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-21 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~stack] I created HBASE-18860 to address where to move this file (and where to 
put all of the new ReadReplica files).

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Affects Versions: HBASE-18477
>Reporter: Zach York
>Assignee: Zach York
> Fix For: HBASE-18477
>
> Attachments: 
> 0001-HBASE-18773-Add-utility-method-to-determine-if-a-Tab.patch, 
> HBASE-18773.HBASE-18477.001.patch, HBASE-18773.HBASE-18477.002.patch, 
> HBASE-18773.HBASE-18477.003.patch, HBASE-18773.HBASE-18477.004.patch, 
> HBASE-18773.HBASE-18477.005.patch, HBASE-18773.HBASE-18477.006.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-20 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18773:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 12m 
36s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
 5s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
15s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
21s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
 9s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
26s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
33s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
16s{color} | {color:green} master passed {color} |
| {color:red}-1{color} | {color:red} mvninstall {color} | {color:red}  0m 
13s{color} | {color:red} hbase-common in the patch failed. {color} |
| {color:red}-1{color} | {color:red} compile {color} | {color:red}  0m 
13s{color} | {color:red} hbase-common in the patch failed. {color} |
| {color:red}-1{color} | {color:red} javac {color} | {color:red}  0m 13s{color} 
| {color:red} hbase-common in the patch failed. {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
 9s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:red}-1{color} | {color:red} shadedjars {color} | {color:red}  0m 
52s{color} | {color:red} patch has 16 errors when building our shaded 
downstream artifacts. {color} |
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  1m 
33s{color} | {color:red} The patch causes 16 errors with Hadoop v2.6.1. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  2m 
14s{color} | {color:red} The patch causes 16 errors with Hadoop v2.6.2. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  2m 
54s{color} | {color:red} The patch causes 16 errors with Hadoop v2.6.3. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  3m 
34s{color} | {color:red} The patch causes 16 errors with Hadoop v2.6.4. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  4m 
14s{color} | {color:red} The patch causes 16 errors with Hadoop v2.6.5. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  4m 
55s{color} | {color:red} The patch causes 16 errors with Hadoop v2.7.1. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  5m 
34s{color} | {color:red} The patch causes 16 errors with Hadoop v2.7.2. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  6m 
14s{color} | {color:red} The patch causes 16 errors with Hadoop v2.7.3. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  6m 
54s{color} | {color:red} The patch causes 16 errors with Hadoop v3.0.0-alpha4. 
{color} |
| {color:red}-1{color} | {color:red} findbugs {color} | {color:red}  0m 
10s{color} | {color:red} hbase-common in the patch failed. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
15s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  0m 14s{color} 
| {color:red} hbase-common in the patch failed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 7s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | 

[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-20 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~stack] Yeah it didn't seem to belong in hbase-common, but I wasn't quite sure 
where to put it. Let's address it afterwards.

This is developing against the https://github.com/apache/hbase/tree/HBASE-18477 
branch (feature branch).

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: 
> 0001-HBASE-18773-Add-utility-method-to-determine-if-a-Tab.patch, 
> HBASE-18773.HBASE-18477.001.patch, HBASE-18773.HBASE-18477.002.patch, 
> HBASE-18773.HBASE-18477.003.patch, HBASE-18773.HBASE-18477.004.patch, 
> HBASE-18773.HBASE-18477.005.patch, HBASE-18773.HBASE-18477.006.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-20 Thread stack (JIRA)

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

stack commented on HBASE-18773:
---

Ugh. My fault. Sorry for the neglect [~zyork].

Let me commit this though I have issue w/ it.

Issues can be addressed in follow-on.

IMO, ReadReplicaClustersTableNameUtil does not belong in hbase-common. 
ReadReplicaClusters is an exotic feature that should be well-encapsulated 
minimally inside a package or in its own module even (might not be enough 
substance to warrant its own module). Perhaps there is another place this class 
might go? No biggie.



> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch, HBASE-18773.HBASE-18477.005.patch, 
> HBASE-18773.HBASE-18477.006.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-18 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~stack] Sorry for the relentless pinging I know you are busy with the 2.0.0 
release :)

Can you take another look?

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch, HBASE-18773.HBASE-18477.005.patch, 
> HBASE-18773.HBASE-18477.006.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-14 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18773:
---

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
14s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
44s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
19s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
27s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
21s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
39s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
22s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
18s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
18s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
22s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
 9s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
34m  4s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
44s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
23s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 9s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 49m 17s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.11.2 Server=1.11.2 Image:yetus/hbase:5d60123 |
| JIRA Issue | HBASE-18773 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12887154/HBASE-18773.HBASE-18477.006.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux a103675146b9 3.13.0-129-generic #178-Ubuntu SMP Fri Aug 11 
12:48:20 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | HBASE-18477 / 8e866d6 |
| Default Java | 1.8.0_144 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8631/testReport/ |
| modules | C: hbase-common U: hbase-common |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8631/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> 

[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-14 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~uagashe] I made the fix in the latest patch.

[~stack] What do you think of the latest patch?

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch, HBASE-18773.HBASE-18477.005.patch, 
> HBASE-18773.HBASE-18477.006.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-14 Thread Umesh Agashe (JIRA)

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

Umesh Agashe commented on HBASE-18773:
--

+1 non-binding. One nit posted on reviewboard. Thanks for the patch [~zyork]!

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-14 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~stack] [~uagashe] [~tedyu] Ping? 

I'm getting ready to put up another patch depending on this and would love to 
be able to rebase on top of this commit (and avoid some complications with 
dev-support/submit-patch which only allows 1 commit to be present)

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-13 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~stack] [~uagashe] [~tedyu] Any other concerns with the patch?

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-12 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18773:
---

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
36s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 14m 
 0s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
47s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
47s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
36s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  1m 
30s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
45s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
46s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
25s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
87m 53s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  2m  
4s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
43s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  4m 
22s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
21s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}118m  0s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.11.2 Server=1.11.2 Image:yetus/hbase:5d60123 |
| JIRA Issue | HBASE-18773 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12886734/HBASE-18773.HBASE-18477.002.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 8a2a552f96db 3.13.0-129-generic #178-Ubuntu SMP Fri Aug 11 
12:48:20 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build@2/component/dev-support/hbase-personality.sh
 |
| git revision | HBASE-18477 / 8e866d6 |
| Default Java | 1.8.0_144 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8590/testReport/ |
| modules | C: hbase-common U: hbase-common |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8590/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> 

[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-12 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18773:
---

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
13s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  8m 
 4s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
20s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
27s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
25s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
38s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
23s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
20s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
24s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
37m 25s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
19s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
10s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 53m  1s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.11.2 Server=1.11.2 Image:yetus/hbase:5d60123 |
| JIRA Issue | HBASE-18773 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12886749/HBASE-18773.HBASE-18477.004.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 4147040561d4 3.13.0-129-generic #178-Ubuntu SMP Fri Aug 11 
12:48:20 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build@2/component/dev-support/hbase-personality.sh
 |
| git revision | HBASE-18477 / 8e866d6 |
| Default Java | 1.8.0_144 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8592/testReport/ |
| modules | C: hbase-common U: hbase-common |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8592/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> 

[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-12 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18773:
---

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
20s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
22s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
16s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
24s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
21s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
31s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
19s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
20s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
 9s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
33m 34s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
40s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
20s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
10s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 47m 52s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:5d60123 |
| JIRA Issue | HBASE-18773 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12886735/HBASE-18773.HBASE-18477.003.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 4c1dccb94df5 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 
12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | HBASE-18477 / 8e866d6 |
| Default Java | 1.8.0_144 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8591/testReport/ |
| modules | C: hbase-common U: hbase-common |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8591/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> 

[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-12 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

Added patch 004 to address issues [~tedyu] brought up in 
https://reviews.apache.org/r/62260/.

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch, 
> HBASE-18773.HBASE-18477.004.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-12 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~stack] I added a new patch that incorporates these comments.

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch, 
> HBASE-18773.HBASE-18477.002.patch, HBASE-18773.HBASE-18477.003.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-08 Thread stack (JIRA)

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

stack commented on HBASE-18773:
---

Sounds good.

Can you corral isMetaTableNameWithSuffix so it is hard-to-find unless you are 
doing this Read Replica Cluster feature -- put it in a utility class or 
whatever (no harm if not as long as it has good javadoc... to distingush it 
from 'normal' function).

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-08 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

bq. Yeah. That looks about right. Make them the same.

[~stack] I think there is some miscommunication here. While I think that making 
a dynamic way of accessing this makes sense, I also need to have both 
implementations of this function. (1) One that checks that this is the correct 
meta table name EXACTLY (including meta suffix if applicable) and (2) one that 
checks if it is a meta table (without any suffixes applied).

I can change the funtionality of isMeta() and isMetaTableName() to (1) and then 
create a new method isMetaTableNameWithSuffix() for (2). What do you think of 
this approach?

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-07 Thread stack (JIRA)

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

stack commented on HBASE-18773:
---

bq. The only issue I see with this approach is that it might be confusing for 
TableName.isMetaTableName() and tableName.isMeta to return different results.

Yeah. That looks about right. Make them the same.

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-07 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

For some reason RB is not letting me publish my comment. Putting it here for 
now:

[~uagashe], yes the HRegionInfo methods used this new method I created in the 
implementation (in a separate patch).


[~stack], do you mean something like what I have below (dynamic isMeta):

public boolean isMeta() {
String[] parts = 
getNameWithNamespaceInclAsString().split(String.valueOf(NAMESPACE_DELIM));
if (parts[0].equals(NamespaceDescriptor.SYSTEM_NAMESPACE_NAME_STR) && 
parts[1].contains(DEFAULT_META_TABLE_NAME_STR)) {
  return true;
}
return false;
  }
  
  
The only issue I see with this approach is that it might be confusing for 
TableName.isMetaTableName() and tableName.isMeta to return different results.

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-07 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18773:
---

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 17m 
31s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 1 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 
47s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
18s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
28s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
20s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
39s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
24s{color} | {color:green} HBASE-18477 passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
19s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
24s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
11s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
35m 29s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
39s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
17s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
40s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
11s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 68m 18s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.0 Server=1.12.0 Image:yetus/hbase:5d60123 |
| JIRA Issue | HBASE-18773 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12885925/HBASE-18773.HBASE-18477.001.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux fdf626544ff7 3.13.0-123-generic #172-Ubuntu SMP Mon Jun 26 
18:04:35 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | HBASE-18477 / 8e866d6 |
| Default Java | 1.8.0_144 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8504/testReport/ |
| modules | C: hbase-common U: hbase-common |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8504/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a 

[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-07 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

[~uagashe] Ah, I originally did the dev on branch-1 where this method doesn't 
exist.

However this is an interesting question. I think it might make sense to have 
two methods:

1. is the TableName this cluster's meta table
2. is the TableName any cluster's meta table (i.e. with any suffix - see 
HBASE-18444)

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-07 Thread Umesh Agashe (JIRA)

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

Umesh Agashe commented on HBASE-18773:
--

[~zyork], there is existing method
{code}
public static boolean isMetaTableName(final TableName tn)
{code}

Does new method serve different purpose?


> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18773) Add utility to determine if a TableName is a meta table

2017-09-07 Thread Zach York (JIRA)

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

Zach York commented on HBASE-18773:
---

The reason this is broken out into a separate commit is because multiple JIRAs 
will depend on this (the first of which being HBASE-18775)

> Add utility to determine if a TableName is a meta table
> ---
>
> Key: HBASE-18773
> URL: https://issues.apache.org/jira/browse/HBASE-18773
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Zach York
>Assignee: Zach York
> Attachments: HBASE-18773.HBASE-18477.001.patch
>
>
> HBASE-18444 adds a method of specifying a meta table suffix. To continue work 
> on HBASE-18477, we need a way to determine if a TableName is a meta table. 
> This patch adds this method and a unit test.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)