[jira] [Commented] (SENTRY-1593) Implement client failover for Generic and NN clients
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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
[ 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)