[GitHub] [incubator-livy] namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328922645 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -161,12 +179,12 @@

[GitHub] [incubator-livy] namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328922002 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -91,29 +95,42 @@

[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-535794176 @yiheng Thanks for your review, Could you also help me check why the test failed? I did not create

[GitHub] [incubator-livy] namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328918752 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -91,29 +95,42 @@

[GitHub] [incubator-livy] zjffdu commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

zjffdu commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328918590 ## File path: repl/scala-2.11/src/main/scala/org/apache/livy/repl/SparkInterpreter.scala ## @@

[GitHub] [incubator-livy] namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328917179 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -81,6 +83,8 @@

[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-535766621 Currently I use operationMessage in LivyThriftSeesionManager to achieve this feature, but maybe

[GitHub] [incubator-livy] namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

namanmishra91 commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328915933 ## File path: repl/scala-2.11/src/main/scala/org/apache/livy/repl/SparkInterpreter.scala ##

[GitHub] [incubator-livy] yiheng commented on issue #235: [LIVY-666] Support named interpreter groups

yiheng commented on issue #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#issuecomment-535776965 Do we need to change the Session recovery process? This is an automated

[GitHub] [incubator-livy] yiheng commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

yiheng commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328905858 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -91,29 +95,42 @@ class

[GitHub] [incubator-livy] yiheng commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

yiheng commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328905600 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -147,7 +164,8 @@ class

[GitHub] [incubator-livy] yiheng commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

yiheng commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328905247 ## File path: repl/src/main/scala/org/apache/livy/repl/Session.scala ## @@ -91,29 +95,42 @@ class

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

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

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

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

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

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

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

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

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

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

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

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

[GitHub] [incubator-livy] zjffdu commented on a change in pull request #235: [LIVY-666] Support named interpreter groups

zjffdu commented on a change in pull request #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#discussion_r328892950 ## File path: repl/scala-2.11/src/main/scala/org/apache/livy/repl/SparkInterpreter.scala ## @@

[GitHub] [incubator-livy] Tagar commented on issue #235: [LIVY-666] Support named interpreter groups

Tagar commented on issue #235: [LIVY-666] Support named interpreter groups URL: https://github.com/apache/incubator-livy/pull/235#issuecomment-535698914 Would this need some changes on Zeppelin too? cc @zjffdu Would this feature be useful outside of Zeppelin? (if yes, then

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

Tagar 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-535695854 Is there is a way to disable this output? Something like `spark.ui.showConsoleProgress`=true/false

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

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] captainzmc removed a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

captainzmc removed 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-535413289 fix This is an automated

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

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=h1) Report > Merging

[GitHub] [incubator-livy] captainzmc commented on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

captainzmc commented on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-535413289 fix This is an automated message from

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

yiheng 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_r328486351 ## File path:

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

yiheng 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_r328485585 ## File path:

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

yiheng 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_r328484319 ## File path:

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

yiheng 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_r328483779 ## File path:

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

yiheng 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_r328481177 ## File path:

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

yiheng 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_r328480394 ## File path:

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

yiheng 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_r328480394 ## File path:

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

yiheng 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_r328479777 ## File path:

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

yiheng 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_r328477974 ## File path:

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

yiheng 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_r328477257 ## File path:

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

yiheng 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_r328473937 ## File path:

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

yiheng 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_r328474161 ## File path:

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

yiheng 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_r328473657 ## File path:

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

yiheng 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_r328473818 ## File path:

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

yiheng 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_r328472542 ## File path:

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

yiheng 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_r328472787 ## File path:

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

yiheng 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_r328472542 ## File path:

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

yiheng 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_r328472030 ## File path:

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

yiheng 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_r328471972 ## File path:

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

yiheng 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_r328471839 ## File path:

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

yiheng 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_r328471720 ## File path:

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

yiheng 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_r328471273 ## File path:

[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_r328470888 ## File path:

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

yiheng 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_r328468908 ## File path:

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

yiheng 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_r328469070 ## File path:

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

yiheng 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_r328468908 ## File path:

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

yiheng 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_r328467437 ## File path:

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

yiheng 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_r328463273 ## File path:

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

yiheng 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_r328463122 ## File path:

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

yiheng 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_r328462885 ## File path:

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

yiheng 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_r328462885 ## File path:

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

yiheng 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_r328461900 ## File path: