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

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

2019-11-17 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/6c7df77204a5a7bfb04beb9789253120d8d7db6c?src=pr=desc)
 will **decrease** coverage by `0.06%`.
   > The diff coverage is `67.03%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   - Coverage  67.9%   67.84%   -0.07% 
   - Complexity  939  947   +8 
   
 Files   102  102  
 Lines  5876 5921  +45 
 Branches891  904  +13 
   
   + Hits   3990 4017  +27 
   - Misses 1314 1327  +13 
   - Partials572  577   +5
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.09% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `64.43% <65.51%> (-1.59%)` | `48 <6> (+8)` | |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `32.87% <0%> (-0.46%)` | `11% <0%> (ø)` | |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `79.91% <0%> (ø)` | `45% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[6c7df77...166c2f5](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

2019-11-12 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/6c7df77204a5a7bfb04beb9789253120d8d7db6c?src=pr=desc)
 will **decrease** coverage by `0.03%`.
   > The diff coverage is `68.96%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   - Coverage  67.9%   67.87%   -0.04% 
   - Complexity  939  947   +8 
   
 Files   102  102  
 Lines  5876 5917  +41 
 Branches891  904  +13 
   
   + Hits   3990 4016  +26 
   - Misses 1314 1325  +11 
   - Partials572  576   +4
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.09% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `65.26% <67.46%> (-0.75%)` | `48 <6> (+8)` | |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `52.94% <0%> (-2.95%)` | `7% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `61.11% <0%> (ø)` | `2% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[6c7df77...2b38d51](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

2019-11-12 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/6c7df77204a5a7bfb04beb9789253120d8d7db6c?src=pr=desc)
 will **decrease** coverage by `0.03%`.
   > The diff coverage is `68.96%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   - Coverage  67.9%   67.87%   -0.04% 
   - Complexity  939  947   +8 
   
 Files   102  102  
 Lines  5876 5917  +41 
 Branches891  904  +13 
   
   + Hits   3990 4016  +26 
   - Misses 1314 1325  +11 
   - Partials572  576   +4
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.09% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `65.26% <67.46%> (-0.75%)` | `48 <6> (+8)` | |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `52.94% <0%> (-2.95%)` | `7% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `61.11% <0%> (ø)` | `2% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[6c7df77...2b38d51](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

2019-11-08 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/9042ff5b4a03cd302a884b5a74280d4b476792b7?src=pr=desc)
 will **increase** coverage by `0.02%`.
   > The diff coverage is `68.6%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   + Coverage 67.83%   67.85%   +0.02% 
   - Complexity  937  947  +10 
   
 Files   102  102  
 Lines  5876 5917  +41 
 Branches891  904  +13 
   
   + Hits   3986 4015  +29 
   - Misses 1316 1326  +10 
   - Partials574  576   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.09% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `65.26% <67.46%> (-0.75%)` | `48 <6> (+8)` | |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `32.87% <0%> (-0.46%)` | `11% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `61.11% <0%> (ø)` | `2% <0%> (ø)` | :arrow_down: |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `79.91% <0%> (+1.25%)` | `45% <0%> (+2%)` | :arrow_up: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[9042ff5...f949f66](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

2019-11-08 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/9042ff5b4a03cd302a884b5a74280d4b476792b7?src=pr=desc)
 will **increase** coverage by `0.02%`.
   > The diff coverage is `68.6%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   + Coverage 67.83%   67.85%   +0.02% 
   - Complexity  937  947  +10 
   
 Files   102  102  
 Lines  5876 5917  +41 
 Branches891  904  +13 
   
   + Hits   3986 4015  +29 
   - Misses 1316 1326  +10 
   - Partials574  576   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.09% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `65.26% <67.46%> (-0.75%)` | `48 <6> (+8)` | |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `32.87% <0%> (-0.46%)` | `11% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `61.11% <0%> (ø)` | `2% <0%> (ø)` | :arrow_down: |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `79.91% <0%> (+1.25%)` | `45% <0%> (+2%)` | :arrow_up: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[9042ff5...f949f66](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

2019-10-13 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/0804c8ea8ece67d01ababec616c9ad8e3b15dc9f?src=pr=desc)
 will **increase** coverage by `0.06%`.
   > The diff coverage is `71.42%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   + Coverage 68.45%   68.52%   +0.06% 
   - Complexity  927  934   +7 
   
 Files   100  100  
 Lines  5729 5760  +31 
 Branches870  880  +10 
   
   + Hits   3922 3947  +25 
   - Misses 1247 1249   +2 
   - Partials560  564   +4
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `95.95% <100%> (+0.08%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `66.11% <69.86%> (+0.09%)` | `46 <4> (+6)` | :arrow_up: |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `81.58% <0%> (+0.83%)` | `46% <0%> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `55.88% <0%> (+2.94%)` | `7% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[0804c8e...cecc5bd](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

2019-10-13 Thread GitBox

codecov-io edited a comment on issue #242: [LIVY-336] 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
   > Merging 
[#242](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/0804c8ea8ece67d01ababec616c9ad8e3b15dc9f?src=pr=desc)
 will **increase** coverage by `0.06%`.
   > The diff coverage is `71.42%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/242/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #242  +/-   ##
   
   + Coverage 68.45%   68.52%   +0.06% 
   - Complexity  927  934   +7 
   
 Files   100  100  
 Lines  5729 5760  +31 
 Branches870  880  +10 
   
   + Hits   3922 3947  +25 
   - Misses 1247 1249   +2 
   - Partials560  564   +4
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `95.95% <100%> (+0.08%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `66.11% <69.86%> (+0.09%)` | `46 <4> (+6)` | :arrow_up: |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `81.58% <0%> (+0.83%)` | `46% <0%> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/242/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `55.88% <0%> (+2.94%)` | `7% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=footer).
 Last update 
[0804c8e...cecc5bd](https://codecov.io/gh/apache/incubator-livy/pull/242?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

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

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

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

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

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

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

7 matches

Site Navigation

Mail list logo

Footer information