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

2019-11-21 Thread GitBox
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-557415337 @jerryshao @mgaido91 Could you please review the new update?

[GitHub] [incubator-livy] codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-555813899 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/259?src=pr=h1) Report >

[GitHub] [incubator-livy] codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-555813899 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/259?src=pr=h1) Report >

[GitHub] [incubator-livy] codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-555813899 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/259?src=pr=h1) Report >

[GitHub] [incubator-livy] runzhiwang commented on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
runzhiwang commented on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-557373410 @jerryshao It's not WIP now. I have change the title.

[GitHub] [incubator-livy] runzhiwang edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
runzhiwang edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-557373410 @jerryshao It's not WIP now. I have changed the title.

[GitHub] [incubator-livy] jerryshao commented on issue #259: [LIVY-714][SERVER][WIP] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
jerryshao commented on issue #259: [LIVY-714][SERVER][WIP] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-557372466 Is it still WIP? @runzhiwang

[GitHub] [incubator-livy] codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-555813899 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/259?src=pr=h1) Report >

[GitHub] [incubator-livy] codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-555813899 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/259?src=pr=h1) Report >

[GitHub] [incubator-livy] codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#issuecomment-555813899 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/259?src=pr=h1) Report >

[GitHub] [incubator-livy] jerryshao commented on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn

2019-11-21 Thread GitBox
jerryshao commented on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-557367902 Looks ok from my side, @yiheng please help to review again.

[GitHub] [incubator-livy] jerryshao closed pull request #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala

2019-11-21 Thread GitBox
jerryshao closed pull request #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala URL: https://github.com/apache/incubator-livy/pull/261 This is an automated message from the Apache

[GitHub] [incubator-livy] codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-538712243 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=h1) Report

[GitHub] [incubator-livy] codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-538712243 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=h1) Report

[GitHub] [incubator-livy] codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-538712243 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=h1) Report

[GitHub] [incubator-livy] codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn

2019-11-21 Thread GitBox
codecov-io edited a comment on issue #242: [LIVY-336][WIP] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#issuecomment-538712243 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=h1) Report

[GitHub] [incubator-livy] Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-21 Thread GitBox
Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending URL: https://github.com/apache/incubator-livy/pull/165#discussion_r349096274 ## File path:

[GitHub] [incubator-livy] Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-21 Thread GitBox
Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending URL: https://github.com/apache/incubator-livy/pull/165#discussion_r349096274 ## File path:

[GitHub] [incubator-livy] Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-21 Thread GitBox
Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending URL: https://github.com/apache/incubator-livy/pull/165#discussion_r349094222 ## File path:

[GitHub] [incubator-livy] Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-21 Thread GitBox
Limmen commented on a change in pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending URL: https://github.com/apache/incubator-livy/pull/165#discussion_r349094222 ## File path:

[GitHub] [incubator-livy] yiheng commented on a change in pull request #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout

2019-11-21 Thread GitBox
yiheng commented on a change in pull request #259: [LIVY-714][SERVER] Fix cannot remove the app in leakedAppTags when timeout URL: https://github.com/apache/incubator-livy/pull/259#discussion_r348946192 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp.scala

[GitHub] [incubator-livy] yiheng commented on issue #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala

2019-11-21 Thread GitBox
yiheng commented on issue #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala URL: https://github.com/apache/incubator-livy/pull/261#issuecomment-556968311 Yes. This template configuration is conflicted with this code