[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309885071 ## File path: server/src/main/scala/org/apache/livy/server/WebServer.scala ## @@

[GitHub] [incubator-livy] ajbozarth commented on issue #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on issue #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#issuecomment-517458899 Also @m-wcislo would you be willing to read through our discussion here and provide some insight? You originally

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309896889 ## File path: server/src/main/scala/org/apache/livy/server/WebServer.scala ## @@

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309888447 ## File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala ## @@

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309892553 ## File path: server/src/main/scala/org/apache/livy/server/WebServer.scala ## @@

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309891166 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ## @@ -60,7

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309886266 ## File path: server/src/main/scala/org/apache/livy/server/WebServer.scala ## @@

[GitHub] [incubator-livy] o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517193430 What IDEA plugin do you use for import ordering? This is an automated

[GitHub] [incubator-livy] jerryshao commented on issue #191: [MINOR] Remove unused guava import

2019-08-01 Thread GitBox
jerryshao commented on issue #191: [MINOR] Remove unused guava import URL: https://github.com/apache/incubator-livy/pull/191#issuecomment-517204487 CC @runzhiwang This is an automated message from the Apache Git Service. To

[GitHub] [incubator-livy] o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236195 BinaryThriftServerSuite."fetch different data types" looks flaky https://travis-ci.org/apache/incubator-livy/jobs/566398488 ```

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io commented on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
codecov-io commented on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] jerryshao opened a new pull request #191: [MINOR] Remove unused guava import

2019-08-01 Thread GitBox
jerryshao opened a new pull request #191: [MINOR] Remove unused guava import URL: https://github.com/apache/incubator-livy/pull/191 ## What changes were proposed in this pull request? PR #181 removed guava dependency in LivyServer, but it still left unused guava import. Here in this

[GitHub] [incubator-livy] codecov-io commented on issue #191: [MINOR] Remove unused guava import

2019-08-01 Thread GitBox
codecov-io commented on issue #191: [MINOR] Remove unused guava import URL: https://github.com/apache/incubator-livy/pull/191#issuecomment-517157615 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/191?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517193430 What plugin do you use for import ordering? This is an automated message

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

2019-08-01 Thread GitBox
arunmahadevan commented on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-517510038 @jahstreet , looks like if the livy pod restarts, the application link (that points to driverUI) for the interactive

[GitHub] [incubator-livy] gumartinm opened a new pull request #192: [LIVY-620] Spark batch session always ends with success when configuration is master yarn and deploy-mode client

2019-08-01 Thread GitBox
gumartinm opened a new pull request #192: [LIVY-620] Spark batch session always ends with success when configuration is master yarn and deploy-mode client URL: https://github.com/apache/incubator-livy/pull/192 ## What changes were proposed in this pull request? Batch session should

[GitHub] [incubator-livy] gumartinm opened a new pull request #192: [LIVY-620] Spark batch session always ends with success when configuration is master yarn and deploy-mode client

2019-08-01 Thread GitBox
gumartinm opened a new pull request #192: [LIVY-620] Spark batch session always ends with success when configuration is master yarn and deploy-mode client URL: https://github.com/apache/incubator-livy/pull/192 ## What changes were proposed in this pull request? Batch session should

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309954905 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ## @@ -60,7

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309953184 ## File path: server/src/main/scala/org/apache/livy/server/WebServer.scala ## @@

[GitHub] [incubator-livy] gumartinm closed pull request #192: [LIVY-620] Spark batch session always ends with success when configuration is master yarn and deploy-mode client

2019-08-01 Thread GitBox
gumartinm closed pull request #192: [LIVY-620] Spark batch session always ends with success when configuration is master yarn and deploy-mode client URL: https://github.com/apache/incubator-livy/pull/192 This is an

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309953116 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ## @@ -60,7

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309952626 ## File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala ## @@

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309952626 ## File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala ## @@

[GitHub] [incubator-livy] ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
ajbozarth commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309954632 ## File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala ## @@

[GitHub] [incubator-livy] yantzu opened a new pull request #193: [LIVY-621]add dynamic service discovry for thrift server

2019-08-01 Thread GitBox
yantzu opened a new pull request #193: [LIVY-621]add dynamic service discovry for thrift server URL: https://github.com/apache/incubator-livy/pull/193 ## What changes were proposed in this pull request? Add config and implementation to allow publish livy thrift server to zookeeper.

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309976585 ## File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala ## @@

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309977304 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ## @@ -60,7

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309975450 ## File path: server/src/main/scala/org/apache/livy/server/WebServer.scala ## @@

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309977304 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ## @@ -60,7

[GitHub] [incubator-livy] codecov-io commented on issue #193: [LIVY-621]add dynamic service discovry for thrift server

2019-08-01 Thread GitBox
codecov-io commented on issue #193: [LIVY-621]add dynamic service discovry for thrift server URL: https://github.com/apache/incubator-livy/pull/193#issuecomment-517544120 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/193?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #193: [LIVY-621]add dynamic service discovry for thrift server

2019-08-01 Thread GitBox
codecov-io edited a comment on issue #193: [LIVY-621]add dynamic service discovry for thrift server URL: https://github.com/apache/incubator-livy/pull/193#issuecomment-517544120 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/193?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH.

2019-08-01 Thread GitBox
fdeantoni commented on a change in pull request #188: LIVY-615: Set context path using property SERVER_BASE_PATH. URL: https://github.com/apache/incubator-livy/pull/188#discussion_r309980622 ## File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala ## @@

[GitHub] [incubator-livy] o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517356964 I have added API for getting Livy Server address to LivyClient, HttpClient, RSCClient and LivyScalaClient. I don't see a way to forward Livy

[GitHub] [incubator-livy] o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517356964 I have added API for getting Livy Server address to LivyClient, HttpClient, RSCClient and LivyScalaClient. I don't see a way to

[GitHub] [incubator-livy] jerryshao closed pull request #191: [MINOR] Remove unused guava import

2019-08-01 Thread GitBox
jerryshao closed pull request #191: [MINOR] Remove unused guava import URL: https://github.com/apache/incubator-livy/pull/191 This is an automated message from the Apache Git Service. To respond to the message, please log on

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

2019-08-01 Thread GitBox
codecov-io 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-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][WIP]: Full support for Spark on Kubernetes

2019-08-01 Thread GitBox
codecov-io 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-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][WIP]: Full support for Spark on Kubernetes

2019-08-01 Thread GitBox
codecov-io 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-481824707 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/167?src=pr=h1) Report > Merging