[GitHub] [incubator-livy] jerryshao closed pull request #262: [LIVY-717] introduce maven property to set ZooKeeper version

2019-11-25 Thread GitBox
jerryshao closed pull request #262: [LIVY-717] introduce maven property to set ZooKeeper version URL: https://github.com/apache/incubator-livy/pull/262 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-livy] yiheng commented on a change in pull request #242: [LIVY-336][SERVER] Livy should not spawn one thread per job to track the job on Yarn

2019-11-25 Thread GitBox
yiheng commented on a change in pull request #242: [LIVY-336][SERVER] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#discussion_r350083545 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp

[GitHub] [incubator-livy] yiheng commented on a change in pull request #242: [LIVY-336][SERVER] Livy should not spawn one thread per job to track the job on Yarn

2019-11-25 Thread GitBox
yiheng commented on a change in pull request #242: [LIVY-336][SERVER] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#discussion_r350165243 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp

[GitHub] [incubator-livy] yiheng commented on a change in pull request #242: [LIVY-336][SERVER] Livy should not spawn one thread per job to track the job on Yarn

2019-11-25 Thread GitBox
yiheng commented on a change in pull request #242: [LIVY-336][SERVER] Livy should not spawn one thread per job to track the job on Yarn URL: https://github.com/apache/incubator-livy/pull/242#discussion_r350162690 ## File path: server/src/main/scala/org/apache/livy/utils/SparkYarnApp

[GitHub] [incubator-livy] symat edited a comment on issue #262: [LIVY-717] introduce maven property to set ZooKeeper version

2019-11-25 Thread GitBox
symat edited a comment on issue #262: [LIVY-717] introduce maven property to set ZooKeeper version URL: https://github.com/apache/incubator-livy/pull/262#issuecomment-558044949 Thanks for the comments! > even if we set the curator.version properly, the zk version from the dependenc

[GitHub] [incubator-livy] symat commented on issue #262: [LIVY-717] introduce maven property to set ZooKeeper version

2019-11-25 Thread GitBox
symat commented on issue #262: [LIVY-717] introduce maven property to set ZooKeeper version URL: https://github.com/apache/incubator-livy/pull/262#issuecomment-558044949 Thanks for the comments! > even if we set the curator.version properly, the zk version from the dependency of cu