[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

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

[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

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] codecov-io commented on issue #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala

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

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] captainzmc closed pull request #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala

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

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] Limmen commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] codecov-io commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] zhaohaifei closed pull request #260: Branch 0.2

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] runzhiwang commented on issue #260: Branch 0.2

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] Limmen opened a new pull request #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] captainzmc commented on issue #261: [LIVY-715][DOC] The configuration in the template is inconsistent with LivyConf.scala

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] captainzmc opened a new pull request #261: [LIVY-715][DOC]The configuration in the template is inconsistent with LivyConf.scala

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] mgaido91 commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] Limmen commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] 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

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

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

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

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] Limmen commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] zhaohaifei commented on issue #260: Branch 0.2

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] yiheng commented on issue #165: [LIVY-581] Fix edge-case where livy overrides user-provided spark properties instead of appending

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] runzhiwang commented on issue #260: Branch 0.2

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] zhaohaifei commented on issue #260: Branch 0.2

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] zhaohaifei commented on issue #260: Branch 0.2

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

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] runzhiwang commented on issue #260: Branch 0.2

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] runzhiwang commented on issue #260: Branch 0.2

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] 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

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

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] zhaohaifei opened a new pull request #260: Branch 0.2

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??