[GitHub] [incubator-livy] yantzu commented on issue #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package

yantzu commented on issue #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package URL: https://github.com/apache/incubator-livy/pull/196#issuecomment-536890270 This PR LGTM. Looking forward to have it mergered to master, is there other concerns about this PR?

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

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

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

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

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

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_r329616871 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ## @

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[GitHub] [incubator-livy] codecov-io commented on issue #240: [LIVY-541] Generate the unqiue session id when multiple livy servers work at the same time

codecov-io commented on issue #240: [LIVY-541] Generate the unqiue session id when multiple livy servers work at the same time URL: https://github.com/apache/incubator-livy/pull/240#issuecomment-536592263 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/240?src=pr&el=h1) Repor

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

mgaido91 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_r329610645 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImpl

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

mgaido91 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_r329609100 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImpl

[GitHub] [incubator-livy] runzhiwang opened a new pull request #240: [LIVY-541] Generate the unqiue session id when multiple livy servers work at the same time

runzhiwang opened a new pull request #240: [LIVY-541] Generate the unqiue session id when multiple livy servers work at the same time URL: https://github.com/apache/incubator-livy/pull/240 ## What changes were proposed in this pull request? [LIVY-541] Generate the unqiue session id w

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

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_r329540012 ## File path: rsc/src/main/java/org/apache/livy/rsc/RSCClient.java ##

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

huianyi closed pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238 This is an automated message from the Apache Git Service. T

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

huianyi opened a new pull request #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238 ## What changes were proposed in this pull request? When we use beeline to connect with thriftserver, we must open th

[GitHub] [incubator-livy] huianyi edited a comment on issue #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

huianyi edited a comment 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-536483452 > Hi @huianyi ，I have saw the ERROR in your uts. The root cause of your UT failure is: > Can'

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

huianyi 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-536483452 > Hi @huianyi ，I have saw the ERROR in your uts. The root cause of your UT failure is: > Can't find

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

captainzmc 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_r329485621 ## File path: rsc/src/main/java/org/apache/livy/rsc/RSCClient.java ###

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

captainzmc 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-536476557 Hi @huianyi ，I have saw the ERROR in your uts. The root cause of your UT failure is: Can't find

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

captainzmc 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-536476832 Hi @huianyi ，I have saw the ERROR in your uts. The root cause of your UT failure is: Can't find c

[GitHub] [incubator-livy] captainzmc removed a comment on issue #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

captainzmc removed a comment 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-536476557 Hi @huianyi ，I have saw the ERROR in your uts. The root cause of your UT failure is: Can'

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

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_r329444807 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerIm

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

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_r329432750 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp

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

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_r329431829 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp

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

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_r329434810 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp

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

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_r329432160 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp

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

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_r329435696 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp

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

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_r329433134 ## File path: server/src/main/scala/org/apache/livy/server/auth/LdapAuthenticationHandlerImp