[GitHub] [incubator-livy] zhaohaifei opened a new pull request #260: Branch 0.2

2019-11-20 Thread GitBox
zhaohaifei opened a new pull request #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260 Who can tell me how to delete livy task in the starting state? livy task has been in the starting state and can't be deleted. Can livy delete tasks in the starting state??

[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-20 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&el=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-20 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&el=h1) Report

[GitHub] [incubator-livy] runzhiwang commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
runzhiwang commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555924452 @zhaohaifei What is the response when you curl delete in starting state. This is an automated mes

[GitHub] [incubator-livy] runzhiwang commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
runzhiwang commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555926821 @zhaohaifei Could you submit different PRs according to feature rather than one PR containing 9 commits.

[GitHub] [incubator-livy] zhaohaifei commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
zhaohaifei commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555927939 > @zhaohaifei What is the response when you curl delete in starting state. response:jar package can not be found. ![ece062a01485a87e8c269af5b96cba7c](https:

[GitHub] [incubator-livy] zhaohaifei commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
zhaohaifei commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555928944 > Could you submit different PRs according to feature rather than one PR containing 9 commits. Sorry, I am a newcomer, I don't know how to do it.

[GitHub] [incubator-livy] zhaohaifei commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
zhaohaifei commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555929940 > > Could you submit different PRs according to feature rather than one PR containing 9 commits. > > Sorry, I am a newcomer, I don't know how to do it. I

[GitHub] [incubator-livy] runzhiwang commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
runzhiwang commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555950789 Do you want to merge your commits to apache/incubator-livy? If so, create one jira for each feature or bug, then submit one Pull Request for each jira. -

[GitHub] [incubator-livy] yiheng commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-20 Thread GitBox
yiheng commented on issue #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#issuecomment-555951399 We meet the same issue... Why close this patch? --

[GitHub] [incubator-livy] zhaohaifei commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
zhaohaifei commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-555952315 > Do you want to merge your commits to apache/incubator-livy? If so, create one jira for each feature or bug, then submit one Pull Request for each jira. no. I

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

2019-11-20 Thread GitBox
Limmen commented on issue #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#issuecomment-555954257 I closed it because I've asked twice on the mailing-list with no response at all..

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

2019-11-20 Thread GitBox
Limmen commented on issue #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#issuecomment-555955221 https://www.mail-archive.com/user@livy.incubator.apache.org/msg00358.html

[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-20 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&el=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-20 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&el=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-20 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&el=h1) Report

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

2019-11-20 Thread GitBox
Limmen commented on issue #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#issuecomment-555957331 I remember unit tests pass locally, still the CI failed for some reason --

[GitHub] [incubator-livy] mgaido91 commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-20 Thread GitBox
mgaido91 commented on issue #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#issuecomment-555957468 @Limmen the CI was failing as stated in my comment. First we need the CI to pass

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

2019-11-20 Thread GitBox
captainzmc opened a new 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 ## What changes were proposed in this pull request? When I test livy impersonation found that, in

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

2019-11-20 Thread GitBox
captainzmc 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-555985436 @jerryshao Just change the doc to avoid misleading the user. Could please help review it?

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

2019-11-20 Thread GitBox
Limmen opened a new 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 This commit fixes a bug in livy that yarn properties are overridden when hive is enabled in the sp

[GitHub] [incubator-livy] runzhiwang commented on issue #260: Branch 0.2

2019-11-20 Thread GitBox
runzhiwang commented on issue #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260#issuecomment-556001581 @zhaohaifei If you want leave message or ask question, please email u...@livy.incubator.apache.org. And you can close this Pull Request Now. ---

[GitHub] [incubator-livy] zhaohaifei closed pull request #260: Branch 0.2

2019-11-20 Thread GitBox
zhaohaifei closed pull request #260: Branch 0.2 URL: https://github.com/apache/incubator-livy/pull/260 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [incubator-livy] codecov-io commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

2019-11-20 Thread GitBox
codecov-io commented on issue #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#issuecomment-556043125 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/165?src=pr&el=h1)

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

2019-11-20 Thread GitBox
Limmen commented on issue #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#issuecomment-556044791 @mgaido91 CI passes now, it was an issue with scalastyle

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

2019-11-20 Thread GitBox
captainzmc 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 G

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

2019-11-20 Thread GitBox
captainzmc opened a new 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 ## What changes were proposed in this pull request? When I test livy impersonation found that, i

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

2019-11-20 Thread GitBox
codecov-io 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-556889331 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/261?src=pr&el=h1) Report

[GitHub] [incubator-livy] codecov-io edited a comment on issue #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala

2019-11-20 Thread GitBox
codecov-io edited a comment 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-556889331 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/261?src=pr&el=h1) Rep

[GitHub] [incubator-livy] yiheng 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-20 Thread GitBox
yiheng 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_r348914530 ## File path: server/src/main/scala/org/apache/livy/serv

[GitHub] [incubator-livy] yiheng 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-20 Thread GitBox
yiheng 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_r348934926 ## File path: server/src/main/scala/org/apache/livy/serv