fdeantoni opened a new pull request #188: LIVY-615: Set context path using
property SERVER_BASE_PATH.
URL: https://github.com/apache/incubator-livy/pull/188
## What changes were proposed in this pull request?
This pull request is to address
https://issues.apache.org/jira/browse/LIVY
codecov-io commented on issue #188: LIVY-615: Set context path using property
SERVER_BASE_PATH.
URL: https://github.com/apache/incubator-livy/pull/188#issuecomment-516730930
#
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/188?src=pr&el=h1)
Report
> Merging
[#188](https://
jerryshao 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_r309121857
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
jerryshao commented on issue #188: LIVY-615: Set context path using property
SERVER_BASE_PATH.
URL: https://github.com/apache/incubator-livy/pull/188#issuecomment-516771810
@ajbozarth would you please help to review. Thanks!
jerryshao closed pull request #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187
This is an automated message from the Apache Git Service.
jerryshao commented on issue #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#issuecomment-516780130
Reopen the PR to retest it.
This is
shanyu opened a new pull request #187: LIVY-617: Livy session leak on Yarn
when creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187
## What changes were proposed in this pull request?
When creating a session with duplicated name, instead of throw exc
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_r309136571
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
jerryshao 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_r309168871
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
jerryshao commented on issue #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#issuecomment-516813276
Thanks @shanyu for the fix. Would you please add an UT about your fix.
Thanks!
-
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_r309178701
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
jerryshao 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_r309185380
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
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_r309193004
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
jerryshao 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_r309194058
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
o-shevchenko opened a new pull request #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189
## What changes were proposed in this pull request?
Implement Livy Server discovery mechanism based on ZooKeeper. And provide
convenient API for client
shanyu opened a new pull request #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190
## What changes were proposed in this pull request?
Add a new configuration:
livy.server.session
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_r309502923
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
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_r309502923
##
File path: server/src/main/scala/org/apache/livy/server/WebServer.scala
##
@@ -
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
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&el=h1)
Report
> Merging
[#191](https://codecov.io/gh/apache/incub
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 from
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
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 r
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
```
B
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/incubat
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
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&el=h1)
Report
> Merging
[#167](https:/
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&el=h1)
Report
> Merging
[#167](https:/
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&el=h1)
Report
> Merging
[#167](https:/
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 a
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 forward
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
##
@@ -
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
##
@@ -
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
##
@@ -
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
##
@@
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
##
@@ -
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 +60
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 adde
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 e
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 se
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
##
@@
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
##
@@
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 +60
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
##
@@ -
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 automated
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 e
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
##
@@
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 +60
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.
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
##
@@ -
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
##
@@
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 +60
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 +60
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&el=h1)
Report
> Merging
[#193](http
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&el=h1)
Report
> Merging
[#193](https://cod
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
##
@@
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/
o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517642960
Refactored configuration to provide a more flexible mechanism for
configuration:
```
# Livy Server discovery
# ZooKeeper quorum URLs,
o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517642960
Refactored configuration to provide a more flexible mechanism for
configuration, simplified code
```
# Livy Server discovery
#
jahstreet commented on issue #167: [LIVY-588][WIP]: Full support for Spark on
Kubernetes
URL: https://github.com/apache/incubator-livy/pull/167#issuecomment-517658246
>
>
> @jahstreet , looks like if the livy pod restarts, the application link
(that points to driverUI) for the inte
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&el=h1)
Report
> Merging
[#167](https:/
jerryshao commented on issue #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#issuecomment-51726
@shanyu would you mind adding a unit test about it.
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&el=h1)
Report
> Merging
[#167](https:/
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/
jerryshao commented on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517677635
Hi @o-shevchenko thanks a lot for your contribution. It would be better to
have a design doc to describe what you're trying to resolve and what i
o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517688172
Thanks a lot @jerryshao for your comment. I'll prepare documentation about
the solution.
I'm just adding storing of Livy Server address/uri
o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517742896
@jerryshao you can find documentation in the Jira ticket:
https://issues.apache.org/jira/browse/LIVY-616.
Please, let me know if something
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/
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&el=h1)
Report
> Merging
[#189](https://codecov.io/gh/apache/
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 forward
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_r310417468
##
File path: server/src/main/scala/org/apache/livy/LivyConf.scala
##
@@ -60,7 +60
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_r310417435
##
File path: server/src/main/scala/org/apache/livy/server/LivyServer.scala
##
@@
yantzu commented on issue #183: [LIVY-609] LDAP auth for Livy thriftserver
URL: https://github.com/apache/incubator-livy/pull/183#issuecomment-518099735
Hi @dockerzhang, My production deployment needs this feature, could I know
when will you finalize the implementation?
---
z-york commented on issue #142: [LIVY-552][WIP] Add JWTFilter to validate
incoming JWT tokens
URL: https://github.com/apache/incubator-livy/pull/142#issuecomment-519263969
My apologies for such a long lapse here... I will be picking this back up
again.
---
shanyu commented on issue #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#issuecomment-519378457
@jerryshao UT added. Thanks for reviewing it.
--
yiheng opened a new pull request #194:
[LIVY-622][LIVY-623][LIVY-624][LIVY-625][Thrift]Support GetFunctions,
GetSchemas, GetTables, GetColumns in Livy thrift server
URL: https://github.com/apache/incubator-livy/pull/194
## What changes were proposed in this pull request?
In this patch,
yiheng commented on a change in pull request #194:
[LIVY-622][LIVY-623][LIVY-624][LIVY-625][Thrift]Support GetFunctions,
GetSchemas, GetTables, GetColumns in Livy thrift server
URL: https://github.com/apache/incubator-livy/pull/194#discussion_r312032036
##
File path:
thriftserver/
yiheng commented on a change in pull request #194:
[LIVY-622][LIVY-623][LIVY-624][LIVY-625][Thrift]Support GetFunctions,
GetSchemas, GetTables, GetColumns in Livy thrift server
URL: https://github.com/apache/incubator-livy/pull/194#discussion_r312037088
##
File path:
thriftserver/
yiheng commented on issue #194:
[LIVY-622][LIVY-623][LIVY-624][LIVY-625][Thrift]Support GetFunctions,
GetSchemas, GetTables, GetColumns in Livy thrift server
URL: https://github.com/apache/incubator-livy/pull/194#issuecomment-519520916
Unlike spark thrift server, we use spark catalog to fe
yiheng edited a comment on issue #194:
[LIVY-622][LIVY-623][LIVY-624][LIVY-625][Thrift]Support GetFunctions,
GetSchemas, GetTables, GetColumns in Livy thrift server
URL: https://github.com/apache/incubator-livy/pull/194#issuecomment-519520916
Unlike spark thrift server, we use spark catalo
shanyu commented on issue #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#issuecomment-519747156
Done. unit test added. But the CI build seems to failed with irrelevant
errors.
--
jerryshao commented on a change in pull request #190: LIVY-547: Livy kills
session after livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#discussion_r312313231
##
File path: server/src/test/scala/org/apache/livy/session
jerryshao commented on issue #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#issuecomment-519757873
Reopen to trigger again.
---
shanyu opened a new pull request #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190
## What changes were proposed in this pull request?
Add a new configuration:
livy.server.session
jerryshao closed pull request #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190
This is an automated message from the Ap
jerryshao closed pull request #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187
This is an automated message from the Apache Git Service.
shanyu opened a new pull request #187: LIVY-617: Livy session leak on Yarn
when creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187
## What changes were proposed in this pull request?
When creating a session with duplicated name, instead of throw exc
jerryshao commented on a change in pull request #187: LIVY-617: Livy session
leak on Yarn when creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#discussion_r312317539
##
File path: server/src/test/scala/org/apache/livy/sessions/MockSession.sca
jerryshao commented on issue #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#issuecomment-519763441
@yiheng please also take a review. Thanks!
-
jerryshao commented on issue #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#issuecomment-519763513
CC @yiheng
This is an automated mes
jerryshao commented on issue #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#issuecomment-519763661
@yiheng please help to review.
-
jerryshao commented on issue #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#issuecomment-519764652
Because we only get state from RM, so it always return SUCCESS in
yiheng commented on a change in 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#discussion_r312343439
##
File path:
server/src/main/scala/org/
yiheng commented on a change in 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#discussion_r312343978
##
File path:
server/src/main/scala/org/
yiheng commented on issue #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#issuecomment-519798443
LGTM.
This is an autom
jerryshao commented on issue #190: LIVY-547: Livy kills session after
livy.server.session.timeout even if the session is active
URL: https://github.com/apache/incubator-livy/pull/190#issuecomment-519800623
Looks like there's some temporary network problems pulling dependences from
maven re
yiheng commented on issue #187: LIVY-617: Livy session leak on Yarn when
creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#issuecomment-519800753
LGTM
This is an automated message from
yiheng removed a comment on issue #187: LIVY-617: Livy session leak on Yarn
when creating session duplicated names
URL: https://github.com/apache/incubator-livy/pull/187#issuecomment-519800753
LGTM
This is an automated messa
801 - 900 of 3662 matches
Mail list logo