[GitHub] [incubator-livy] huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333841848 ## File path: rsc/src/main/java/org/apache/livy/rsc/BaseProtocol.java ###

[GitHub] [incubator-livy] codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-10 Thread GitBox
codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#issuecomment-534468593 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/231?src=pr&el=h1) Report > Merging [#231](ht

[GitHub] [incubator-livy] codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-10 Thread GitBox
codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#issuecomment-534468593 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/231?src=pr&el=h1) Report > Merging [#231](ht

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333809766 ## File path: rsc/src/main/java/org/apache/livy/rsc/driver/RSCDriver.java #

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333808458 ## File path: core/src/main/scala/org/apache/livy/ConcurrentBoundedLinkedQu

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333808344 ## File path: core/src/main/scala/org/apache/livy/ConcurrentBoundedLinkedQu

[GitHub] [incubator-livy] jerryshao commented on issue #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on issue #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#issuecomment-540876001 > Beeline from Hive 2.2 has in-place progress bar hive.server2.in.place.progress > >https://cw

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333806025 ## File path: rsc/src/main/java/org/apache/livy/rsc/driver/JobWrapper.java

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333805483 ## File path: rsc/src/main/java/org/apache/livy/rsc/BaseProtocol.java #

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333805180 ## File path: core/src/main/scala/org/apache/livy/OperationMessageManager.sc

[GitHub] [incubator-livy] jerryshao closed pull request #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-10 Thread GitBox
jerryshao closed pull request #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-livy] peter-toth opened a new pull request #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-10 Thread GitBox
peter-toth opened a new pull request #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243 ## What changes were proposed in this pull request? Upgrade Jackson to 2.9.10 which fixes many CVEs: https://github.com/FasterXML/jackson/wiki/Jackso

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-10 Thread GitBox
jerryshao commented on a change in pull request #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#discussion_r333802329 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333407101 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333407101 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333407101 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

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

2019-10-10 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&el=h1) Report > Mergi

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

2019-10-10 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&el=h1) Report > Mergi

[GitHub] [incubator-livy] codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-10 Thread GitBox
codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#issuecomment-534468593 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/231?src=pr&el=h1) Report > Merging [#231](ht

[GitHub] [incubator-livy] codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-10 Thread GitBox
codecov-io edited a comment on issue #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#issuecomment-534468593 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/231?src=pr&el=h1) Report > Merging [#231](ht

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

2019-10-10 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_r333422091 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333421978 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333424915 ## File path: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333424085 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333423403 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333422091 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333421978 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333420717 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333420717 ## File path: thriftserver/server/src/test/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333419265 ## File path: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/

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

2019-10-10 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_r333418190 ## File path: thriftserver/server/src/main/scala/org/apache/livy/thriftserver/

[GitHub] [incubator-livy] captainzmc commented on a change in pull request #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server.

2019-10-10 Thread GitBox
captainzmc commented on a change in pull request #231: [LIVY-356][SERVER]Add LDAP authentication for livy-server. URL: https://github.com/apache/incubator-livy/pull/231#discussion_r333415057 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerIm

[GitHub] [incubator-livy] huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r333412835 ## File path: core/src/main/scala/org/apache/livy/OperationMessageManager.scal

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333409622 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333408607 ## File path: conf/livy.conf.template ## @@ -121,15 +121,21 @@

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333408762 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333407101 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333403822 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r333403822 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] runzhiwang commented on a change in pull request #242: [LIVY-336] Livy should not spawn one thread per job to track the job on Yarn

2019-10-10 Thread GitBox
runzhiwang commented on a change in pull request #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#discussion_r81184 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.sca

[GitHub] [incubator-livy] yiheng commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
yiheng commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r78399 ## File path: core/src/main/scala/org/apache/livy/OperationMessageManager.scala

[GitHub] [incubator-livy] huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r68731 ## File path: core/src/main/scala/org/apache/livy/OperationMessageManager.scal

[GitHub] [incubator-livy] huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-10 Thread GitBox
huianyi commented on a change in pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#discussion_r68731 ## File path: core/src/main/scala/org/apache/livy/OperationMessageManager.scal