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

2019-08-02 Thread GitBox
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

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

2019-08-02 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 edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 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] o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 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] codecov-io edited a comment on issue #167: [LIVY-588][WIP]: Full support for Spark on Kubernetes

2019-08-02 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] jerryshao commented on issue #190: LIVY-547: Livy kills session after livy.server.session.timeout even if the session is active

2019-08-02 Thread GitBox
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.

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

2019-08-02 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] o-shevchenko edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 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-517642960 Refactored configuration to provide a more flexible mechanism for configuration, simplified code ``` # Livy Server discovery

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

2019-08-02 Thread GitBox
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

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

2019-08-02 Thread GitBox
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

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

2019-08-02 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 edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 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 edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 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] o-shevchenko commented on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 Thread GitBox
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

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

2019-08-02 Thread GitBox
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