[GitHub] [incubator-livy] mgaido91 commented on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
mgaido91 commented on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541396991 @jahstreet I noticed this is still WIP and it has been so for a long. Are you still working on this?

[GitHub] [incubator-livy] jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 @mgaido91 Hey, actually feels like it is done, the only problem with testing I faced is that my implicit extension for

[GitHub] [incubator-livy] mgaido91 closed pull request #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-13 Thread GitBox
mgaido91 closed pull request #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-livy] mgaido91 commented on issue #245: [LIVY-695] Upgrade JQuery to 3.4.1 and Bootstrap to 3.4.1

2019-10-13 Thread GitBox
mgaido91 commented on issue #245: [LIVY-695] Upgrade JQuery to 3.4.1 and Bootstrap to 3.4.1 URL: https://github.com/apache/incubator-livy/pull/245#issuecomment-541396584 cc @ajbozarth This is an automated message from the

[GitHub] [incubator-livy] jahstreet commented on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet commented on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 > @jahstreet I noticed this is still WIP and it has been so for a long. Are you still working on this? Hey,

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] ajbozarth commented on issue #245: [LIVY-695] Upgrade JQuery to 3.4.1 and Bootstrap to 3.4.1

2019-10-13 Thread GitBox
ajbozarth commented on issue #245: [LIVY-695] Upgrade JQuery to 3.4.1 and Bootstrap to 3.4.1 URL: https://github.com/apache/incubator-livy/pull/245#issuecomment-541398219 Is there a reason you added 8 new bootstrap files? When downloading bootstrap it allows you to pick and choose which

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 @mgaido91 Hey, actually feels like it is done, the only problem with testing I faced is that my implicit extension for

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] mgaido91 commented on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-13 Thread GitBox
mgaido91 commented on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#issuecomment-541395276 LGTM as well. Thanks, merging to master. This is an

[GitHub] [incubator-livy] jahstreet edited a comment on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet edited a comment on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 @mgaido91 Hey, actually left only to write test. Last month had no time, plan to finish tests this month and it

[GitHub] [incubator-livy] peter-toth commented on issue #245: [LIVY-695] Upgrade JQuery to 3.4.1 and Bootstrap to 3.4.1

2019-10-13 Thread GitBox
peter-toth commented on issue #245: [LIVY-695] Upgrade JQuery to 3.4.1 and Bootstrap to 3.4.1 URL: https://github.com/apache/incubator-livy/pull/245#issuecomment-541400372 Thanks @ajbozarth for the review. I will remove the unused files then.

[GitHub] [incubator-livy] jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 @mgaido91 Hey, actually feels like it is done, the only problem with testing I faced is that my implicit extension for

[GitHub] [incubator-livy] mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r334267285 ## File path:

[GitHub] [incubator-livy] mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r334267214 ## File path:

[GitHub] [incubator-livy] codecov-io edited a comment on issue #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-538712243 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=h1) Report >

[GitHub] [incubator-livy] codecov-io edited a comment on issue #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-538712243 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=h1) Report >

[GitHub] [incubator-livy] mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r334267205 ## File path:

[GitHub] [incubator-livy] mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r334266985 ## File path:

[GitHub] [incubator-livy] mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r334267227 ## File path:

[GitHub] [incubator-livy] peter-toth commented on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429

2019-10-13 Thread GitBox
peter-toth commented on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/incubator-livy/pull/244#issuecomment-541397716 Thanks for letting me know about that issue. I will check it with the updated Jetty version.

[GitHub] [incubator-livy] codecov-io edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-481824707 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/167?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
codecov-io edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-481824707 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/167?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] captainzmc commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-13 Thread GitBox
captainzmc commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r334270376 ## File path:

[GitHub] [incubator-livy] jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet edited a comment on issue #167: [LIVY-588]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 @mgaido91 Hey, actually feels like it is done, the only problem with testing I faced is that my implicit extension for

[GitHub] [incubator-livy] jahstreet edited a comment on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes

2019-10-13 Thread GitBox
jahstreet edited a comment on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-541399267 @mgaido91 Hey, actually feels like it is done, the only problem with testing I faced is that my implicit extension

[GitHub] [incubator-livy] yantzu commented on issue #195: [LIVY-634] To support reuse hive PasswdAuthenticationProvider implementation in ThriftServer

2019-10-13 Thread GitBox
yantzu commented on issue #195: [LIVY-634] To support reuse hive PasswdAuthenticationProvider implementation in ThriftServer URL: https://github.com/apache/incubator-livy/pull/195#issuecomment-541483972 close this PR as there is better solution.

[GitHub] [incubator-livy] yantzu closed pull request #195: [LIVY-634] To support reuse hive PasswdAuthenticationProvider implementation in ThriftServer

2019-10-13 Thread GitBox
yantzu closed pull request #195: [LIVY-634] To support reuse hive PasswdAuthenticationProvider implementation in ThriftServer URL: https://github.com/apache/incubator-livy/pull/195 This is an automated message from the