[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-27 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15988054#comment-15988054
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865418/SENTRY-1593.011-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:green}Overall:{color} +1 all checks pass

{color:green}SUCCESS:{color} all tests passed

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2599/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> SENTRY-1593.011-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-27 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15987921#comment-15987921
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865418/SENTRY-1593.011-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryWebServerWithSSL

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2597/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> SENTRY-1593.011-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-27 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15987518#comment-15987518
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865418/SENTRY-1593.011-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryWebServerWithSSL

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2592/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> SENTRY-1593.011-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-27 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15987364#comment-15987364
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865411/SENTRY-1593.011-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 3 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2590/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> SENTRY-1593.011-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15984454#comment-15984454
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865056/SENTRY-1593.010-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 3 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2575/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15984379#comment-15984379
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865056/SENTRY-1593.010-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:green}Overall:{color} +1 all checks pass

{color:green}SUCCESS:{color} all tests passed

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2572/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-25 Thread kalyan kumar kalvagadda (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15984085#comment-15984085
 ] 

kalyan kumar kalvagadda commented on SENTRY-1593:
-

This is passed locally.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-25 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15984045#comment-15984045
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12865036/SENTRY-1593.010-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryWebServerWithSSL

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2571/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> SENTRY-1593.010-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-19 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15975374#comment-15975374
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12864069/SENTRY-1593.009-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} failed to apply patch (exit code 1):
The patch does not appear to apply with p0, p1, or p2



Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2539/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-19 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15975035#comment-15975035
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12864040/SENTRY-1593.009-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} failed to apply patch (exit code 1):
The patch does not appear to apply with p0, p1, or p2



Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2538/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-19 Thread kalyan kumar kalvagadda (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15975028#comment-15975028
 ] 

kalyan kumar kalvagadda commented on SENTRY-1593:
-

[~akolb] Attached the latest class diagram

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, 
> service_client_class_diagram_latest_v1.png, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-18 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15973711#comment-15973711
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12863906/SENTRY-1593.009-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} failed to apply patch (exit code 1):
The patch does not appear to apply with p0, p1, or p2



Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2533/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.009-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-17 Thread Alexander Kolbasov (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15971910#comment-15971910
 ] 

Alexander Kolbasov commented on SENTRY-1593:


[~kkalyan] The class diagram probably should be updated based on the new 
changes.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-14 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969383#comment-15969383
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12863490/SENTRY-1593.007-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.persistent.TestPrivilegeOperatePersistence

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2523/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-14 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969103#comment-15969103
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12863471/SENTRY-1593.008-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryWebServerWithSSL

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2520/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.008-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-13 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15968407#comment-15968407
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12863398/SENTRY-1593.008-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 3 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.persistent.TestSentryStore

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2517/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.008-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-13 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15968259#comment-15968259
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12863368/SENTRY-1593.008-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryWebServerWithSSL

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2514/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, 
> SENTRY-1593.007-sentry-ha-redesign.patch, 
> SENTRY-1593.008-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-04-06 Thread Alexander Kolbasov (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15959800#comment-15959800
 ] 

Alexander Kolbasov commented on SENTRY-1593:


I would suggest a bit different class structure. For the base class used by 
various default implementations:

{code}
/**
 * General representation of transport connection to Sentry
 */
public interface SentrySocket extends AutoCloseable {
  /**
   * Connect to the Sentry server
   * @throws IOException
   */
  void connect() throws IOException;
}
{code}

Specific implementations would loke like this:

{code}
public class SentryGenericServiceClientDefaultImpl
implements SentryGenericServiceClient, SentrySocket {
{code}

The SentryServiceClientTransportDefaultImpl can be renamed to a stand-alone 
class

{code}
**
 * Generate Thrift transports suitable for talking to Sentry
 */
public final class SentryTransportFactory {
...
  public TTransport connect() throws IOException {
   // Connect to one of multiple servers
  }
{code}

Specific impl classes will the do

{code}
  private final SentryTransportFactory transportFactory;
 ...
// Constructor
transportFactory = new SentryTransportFactory(conf, transportConfig);
...
  @Override
  public void connect() throws IOException {
if (isOpen()) {
  return;
}

transport = transportFactory.connect();

...

  @Override
  public void close() {
transport.close();
  }
{code}

And the RetryClientInvocationHandler will do

{code}
while (retryCount < retries) {
  // Connect to a sentry server if not connected yet.
  try {
client.connect();
  } catch (IOException e) {
// Increase the retry num
// Retry when the exception is caused by connection problem.
retryCount++;
client.close();
continue;
  }
...
{code}

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>Priority: Blocker
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-31 Thread kalyan kumar kalvagadda (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15951706#comment-15951706
 ] 

kalyan kumar kalvagadda commented on SENTRY-1593:
-

I ran the test TestDbCrossDbOps and is the test passed.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-31 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15951457#comment-15951457
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12861474/SENTRY-1593.006-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 2 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.tests.e2e.dbprovider.TestDbCrossDbOps

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2463/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, 
> SENTRY-1593.006-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-31 Thread kalyan kumar kalvagadda (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15951294#comment-15951294
 ] 

kalyan kumar kalvagadda commented on SENTRY-1593:
-

At a high level the change set submitted does following.

1. Extends the capability of sentry name-node client and generic policy client 
to connect to multiple servers and failover to the server that is available
2. Transport handling of the clients is abstracted to another class so that the 
client's just have to extend the class that abstracts it. Implementations of 
the clients don't have to worry about it.
3. Changed the behavior of the clients so that they connect to one of the 
configured servers when the client is actually instantiated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-30 Thread Alexander Kolbasov (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15949804#comment-15949804
 ] 

Alexander Kolbasov commented on SENTRY-1593:


[~kkalyan] It is very difficult to review such big patch which involves moving 
files around as well as changing them. Can you refactor the patch in two parts:

1) Moving all files around as you need without changing them (other then the 
changes necessary for the move)
2) Actually changing files for the feature.

The first one should be a very straightforward code review, the second can 
concentrate on the actual meat of the change.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-29 Thread Alexander Kolbasov (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15948442#comment-15948442
 ] 

Alexander Kolbasov commented on SENTRY-1593:


[~kkalyan] Can you add a description of your changes here?

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-29 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15947317#comment-15947317
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12861057/SENTRY-1593.005-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 3 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.persistent.TestPrivilegeOperatePersistence
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryPolicyServiceClient

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2458/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, 
> SENTRY-1593.004-sentry-ha-redesign.patch, 
> SENTRY-1593.005-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-03-23 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15939493#comment-15939493
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12860250/SENTRY-1593.004-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 68 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryWebServerWithSSL
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceWithInvalidMsgSize
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceWithInvalidMsgSize
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceClientPool
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceClientPool
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceForPoolWithKerberos
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellKafka
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestAuditLogForSentryGenericService
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.tools.TestSentryShellSolr
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryPolicyServiceClient
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceWithKerberos
{color:red}ERROR:{color} Failed: 

[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-02-20 Thread Alexander Kolbasov (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15874727#comment-15874727
 ] 

Alexander Kolbasov commented on SENTRY-1593:


The patch is becoming rather big. I suggest splitting the code refactoring  
(moving configuration for all three clients into a common place) into a 
separate JIRA/subtask - this will simplify various merges, etc.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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


[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients

2017-02-10 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/SENTRY-1593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15861853#comment-15861853
 ] 

Hadoop QA commented on SENTRY-1593:
---

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12852114/SENTRY-1593.003-sentry-ha-redesign.patch
 against sentry-ha-redesign.

{color:red}Overall:{color} -1 due to 20 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestAuditLogForSentryGenericService
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceFailureCase
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.generic.service.thrift.TestSentryGenericServiceIntegration
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceClientPool
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceClientPool
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestAuthorizingDDLAuditLogWithKerberos
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceWithInvalidMsgSize
{color:red}ERROR:{color} Failed: 
org.apache.sentry.provider.db.service.thrift.TestSentryServiceWithInvalidMsgSize
{color:red}ERROR:{color} Failed: 
org.apache.sentry.tests.e2e.dbprovider.TestDbPrivilegesAtFunctionScope

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/2332/console

This message is automatically generated.

> Implement client failover for Generic and NN clients
> 
>
> Key: SENTRY-1593
> URL: https://issues.apache.org/jira/browse/SENTRY-1593
> Project: Sentry
>  Issue Type: Sub-task
>  Components: Sentry
>Affects Versions: sentry-ha-redesign
>Reporter: Alexander Kolbasov
>Assignee: kalyan kumar kalvagadda
>  Labels: HA
> Fix For: sentry-ha-redesign
>
> Attachments: old_service_client_class_diagram.png, 
> SENTRY-1593.001-sentry-ha-redesign.patch, 
> SENTRY-1593.002-sentry-ha-redesign.patch, 
> SENTRY-1593.003-sentry-ha-redesign.patch, service_client_class_diagram.png
>
>
> We need to have client failover logic for Generic service clients and Name 
> Node clients. Currently only db policy clients have it implemented.



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