[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-30 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


ABORTED: Integrated in Jenkins build HBase-HBASE-14614 #190 (See 
[https://builds.apache.org/job/HBase-HBASE-14614/190/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
9a1aff447e908c9de351a4f45b869b016ad7821b)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-28 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.1-JDK7 #1863 (See 
[https://builds.apache.org/job/HBase-1.1-JDK7/1863/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
23a3e755f676f5f11fb5eae935c62ba389487a89)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-28 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.1-JDK8 #1946 (See 
[https://builds.apache.org/job/HBase-1.1-JDK8/1946/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
23a3e755f676f5f11fb5eae935c62ba389487a89)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-25 Thread lv zehui (JIRA)

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

lv zehui commented on HBASE-17514:
--

[~busbey] Thanks for your help. This is really a good start. Let me see what I 
can do next.

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #2904 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/2904/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
9a1aff447e908c9de351a4f45b869b016ad7821b)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.3-JDK7 #150 (See 
[https://builds.apache.org/job/HBase-1.3-JDK7/150/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
1b0ebf8d1ef87ffb4e0f6593b937a34871d0cd45)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.4 #702 (See 
[https://builds.apache.org/job/HBase-1.4/702/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
9d1deeec8c00d86d097d3b4d00d07ff98266ab19)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.3-JDK8 #163 (See 
[https://builds.apache.org/job/HBase-1.3-JDK8/163/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
1b0ebf8d1ef87ffb4e0f6593b937a34871d0cd45)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.2-JDK7 #126 (See 
[https://builds.apache.org/job/HBase-1.2-JDK7/126/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
92eefa388f545406961b4d4dbc113e71960cad9e)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.2-JDK8 #122 (See 
[https://builds.apache.org/job/HBase-1.2-JDK8/122/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
92eefa388f545406961b4d4dbc113e71960cad9e)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.2-IT #861 (See 
[https://builds.apache.org/job/HBase-1.2-IT/861/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
92eefa388f545406961b4d4dbc113e71960cad9e)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Hudson (JIRA)

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

Hudson commented on HBASE-17514:


SUCCESS: Integrated in Jenkins build HBase-1.3-IT #30 (See 
[https://builds.apache.org/job/HBase-1.3-IT/30/])
HBASE-17514 emit a warning if thrift1 proxy user is configured but (busbey: rev 
1b0ebf8d1ef87ffb4e0f6593b937a34871d0cd45)
* (edit) 
hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java


> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-17514:
-

Thanks for the contribution [~zehui.lv]!

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-24 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-17514:
-

{quote}
Oh, I thought resolve means my part is finished and it's time to report. Sorry
{quote}

No worries. Getting you used to our submission process is part of why 
committers are here. :)

{code}
-1  test4tests  0m 0s   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.
{code}

Since this is just a log change and we don't have a way to test log output, no 
need for test changes.

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-22 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-17514:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 32s 
{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:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s 
{color} | {color:red} 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} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m 
24s {color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 20s 
{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 
25s {color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 
15s {color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 0s 
{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 19s 
{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 
21s {color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 20s 
{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 20s 
{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 
14s {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} 
27m 19s {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-alpha2. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 7s 
{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} 1m 31s 
{color} | {color:green} hbase-thrift in the patch passed. {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} | {color:black} {color} | {color:black} 38m 17s {color} 
| {color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:8d52d23 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12864633/HBASE-17514.master.001.patch
 |
| JIRA Issue | HBASE-17514 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux b89ad8bd58b4 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 
09:57:27 UTC 2016 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 | master / 9053ec6 |
| Default Java | 1.8.0_121 |
| findbugs | v3.0.0 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/6538/testReport/ |
| modules | C: hbase-thrift U: hbase-thrift |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/6538/console |
| Powered by | Apache Yetus 0.3.0   http://yetus.apache.org |


This message was automatically generated.



> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  

[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-22 Thread lv zehui (JIRA)

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

lv zehui commented on HBASE-17514:
--

Oh, I thought *resolve* means my part is finished and it's time to report. 
Sorry:(

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Fix For: 2.0.0, 1.4.0, 1.2.6, 1.3.2, 1.1.11
>
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-22 Thread lv zehui (JIRA)

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

lv zehui commented on HBASE-17514:
--

I checked out the 
[doc|https://hbase.apache.org/book.html#security.gateway.thrift.doas]. It looks 
like the doAs feature only works on thrift1.

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-17514.master.001.patch
>
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-21 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-17514:
-

all yours!

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Assignee: lv zehui
>Priority: Minor
>  Labels: beginner
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-20 Thread lv zehui (JIRA)

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

lv zehui commented on HBASE-17514:
--

OK. I can take this one. Could you please assign this jira to me?

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Priority: Minor
>  Labels: beginner
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-20 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-17514:
-

yeah that looks good.

I don't know off the top of my head if the thrift2 server supports proxy users. 
I would guess I only talked about the thrift1 server in this jira because I 
happened to be looking at it at the time. If it looks like thrift2 has the same 
issue, feel free to do both under this issue.

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Priority: Minor
>  Labels: beginner
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HBASE-17514) Warn when Thrift Server 1 is configured for proxy users but not the HTTP transport

2017-04-19 Thread lv zehui (JIRA)

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

lv zehui commented on HBASE-17514:
--

[~busbey] I go to ThriftServerRunner's construstor and emit a warning there.
The code looks like:
{code}
if (doAsEnabled) {
  if (!conf.getBoolean(USE_HTTP_CONF_KEY, false)) {
LOG.warn("");
  }
}
}
{code}
Does it look good to you? How about thrift2?

> Warn when Thrift Server 1 is configured for proxy users but not the HTTP 
> transport
> --
>
> Key: HBASE-17514
> URL: https://issues.apache.org/jira/browse/HBASE-17514
> Project: HBase
>  Issue Type: Improvement
>  Components: Thrift, Usability
>Reporter: Sean Busbey
>Priority: Minor
>  Labels: beginner
>
> The config {{hbase.thrift.support.proxyuser}} is ignored if the Thrift Server 
> 1 isn't configured to use an HTTP transport with 
> {{hbase.regionserver.thrift.http}}.
> We should emit a warning if our configs request proxy user support but don't 
> specify that HTTP should be used for the transport.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)