[GitHub] [incubator-livy] shanyu opened a new pull request #187: LIVY-617: Livy session leak on Yarn when creating session duplicated names

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

[GitHub] [incubator-livy] jerryshao commented on issue #187: LIVY-617: Livy session leak on Yarn when creating session duplicated names

2019-08-15 Thread GitBox
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-521539642 Reopen to test it. This is an

[GitHub] [incubator-livy] shanyu commented on a change in pull request #187: LIVY-617: Livy session leak on Yarn when creating session duplicated names

2019-08-15 Thread GitBox
shanyu 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_r314205285 ## File path:

[GitHub] [incubator-livy] jerryshao closed pull request #198: [LIVY-635][BUILD] Fix travis fail to build by excluding the dependency of hbase

2019-08-15 Thread GitBox
jerryshao closed pull request #198: [LIVY-635][BUILD] Fix travis fail to build by excluding the dependency of hbase URL: https://github.com/apache/incubator-livy/pull/198 This is an automated message from the Apache Git

[GitHub] [incubator-livy] mgaido91 commented on issue #198: [LIVY-635][BUILD] Fix travis fail to build by excluding the dependency of hbase

2019-08-15 Thread GitBox
mgaido91 commented on issue #198: [LIVY-635][BUILD] Fix travis fail to build by excluding the dependency of hbase URL: https://github.com/apache/incubator-livy/pull/198#issuecomment-521741922 LGTM too, this should be safe as AFAIK, dependencies on HBase are needed only for using HBase as

[GitHub] [incubator-livy] jerryshao closed pull request #187: LIVY-617: Livy session leak on Yarn when creating session duplicated names

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

[GitHub] [incubator-livy] shanyu opened a new pull request #187: LIVY-617: Livy session leak on Yarn when creating session duplicated names

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

[GitHub] [incubator-livy] jerryshao commented on issue #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package

2019-08-15 Thread GitBox
jerryshao commented on issue #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package URL: https://github.com/apache/incubator-livy/pull/196#issuecomment-521861706 @yiheng would you please also review this patch, thanks!

[GitHub] [incubator-livy] jerryshao commented on a change in pull request #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package

2019-08-15 Thread GitBox
jerryshao commented on a change in pull request #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package URL: https://github.com/apache/incubator-livy/pull/196#discussion_r314563694 ## File path: conf/livy-client.conf.template ## @@ -67,8

[GitHub] [incubator-livy] jerryshao commented on issue #194: [LIVY-622][LIVY-623][LIVY-624][LIVY-625][Thrift]Support GetFunctions, GetSchemas, GetTables, GetColumns in Livy thrift server

2019-08-15 Thread GitBox
jerryshao 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-521861935 LGTM, merging to master branch, thanks for the