[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-14 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.04%`.
   > The diff coverage is `87.5%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.66%   -0.05% 
   - Complexity  908  909   +1 
   
 Files   100  103   +3 
 Lines  5674 5710  +36 
 Branches854  854  
   
   + Hits   3899 3921  +22 
   - Misses 1219 1231  +12 
   - Partials556  558   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `63.04% <ø> (ø)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.01% <100%> (+0.16%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `80% <100%> (ø)` | `10 <1> (ø)` | :arrow_down: |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `100% <100%> (ø)` | `5 <1> (ø)` | :arrow_down: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | ... and [16 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...18c10ed](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-14 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.04%`.
   > The diff coverage is `87.5%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.66%   -0.05% 
   - Complexity  908  909   +1 
   
 Files   100  103   +3 
 Lines  5674 5710  +36 
 Branches854  854  
   
   + Hits   3899 3921  +22 
   - Misses 1219 1231  +12 
   - Partials556  558   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `63.04% <ø> (ø)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.01% <100%> (+0.16%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `80% <100%> (ø)` | `10 <1> (ø)` | :arrow_down: |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `100% <100%> (ø)` | `5 <1> (ø)` | :arrow_down: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | ... and [16 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...18c10ed](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.01%`.
   > The diff coverage is `82.14%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.69%   -0.02% 
   - Complexity  908  922  +14 
   
 Files   100  103   +3 
 Lines  5674 5712  +38 
 Branches854  853   -1 
   
   + Hits   3899 3924  +25 
   - Misses 1219 1233  +14 
   + Partials556  555   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96% <100%> (+0.14%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `66.66% <71.42%> (-12.41%)` | `6 <5> (-11)` | |
   | 
[...e/livy/server/discovery/LivyDiscoveryManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0xpdnlEaXNjb3ZlcnlNYW5hZ2VyLnNjYWxh)
 | `75% <75%> (ø)` | `4 <4> (?)` | |
   | 
[...pache/livy/server/discovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L1pvb0tlZXBlck1hbmFnZXIuc2NhbGE=)
 | `78.04% <78.04%> (ø)` | `16 <16> (?)` | |
   | ... and [11 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...1d0add4](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.01%`.
   > The diff coverage is `82.14%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.69%   -0.02% 
   - Complexity  908  922  +14 
   
 Files   100  103   +3 
 Lines  5674 5712  +38 
 Branches854  853   -1 
   
   + Hits   3899 3924  +25 
   - Misses 1219 1233  +14 
   + Partials556  555   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96% <100%> (+0.14%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `66.66% <71.42%> (-12.41%)` | `6 <5> (-11)` | |
   | 
[...e/livy/server/discovery/LivyDiscoveryManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0xpdnlEaXNjb3ZlcnlNYW5hZ2VyLnNjYWxh)
 | `75% <75%> (ø)` | `4 <4> (?)` | |
   | 
[...pache/livy/server/discovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L1pvb0tlZXBlck1hbmFnZXIuc2NhbGE=)
 | `78.04% <78.04%> (ø)` | `16 <16> (?)` | |
   | ... and [11 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...1d0add4](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.01%`.
   > The diff coverage is `83.95%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.69%   -0.02% 
   - Complexity  908  924  +16 
   
 Files   100  103   +3 
 Lines  5674 5709  +35 
 Branches854  852   -2 
   
   + Hits   3899 3922  +23 
   - Misses 1219 1232  +13 
   + Partials556  555   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96% <100%> (+0.14%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `38.2% <100%> (+3.23%)` | `15 <4> (+4)` | :arrow_up: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `66.66% <71.42%> (-12.41%)` | `6 <5> (-11)` | |
   | 
[...e/livy/server/discovery/LivyDiscoveryManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0xpdnlEaXNjb3ZlcnlNYW5hZ2VyLnNjYWxh)
 | `75% <75%> (ø)` | `4 <4> (?)` | |
   | ... and [12 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...728deb9](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.01%`.
   > The diff coverage is `83.95%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.69%   -0.02% 
   - Complexity  908  924  +16 
   
 Files   100  103   +3 
 Lines  5674 5709  +35 
 Branches854  852   -2 
   
   + Hits   3899 3922  +23 
   - Misses 1219 1232  +13 
   + Partials556  555   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96% <100%> (+0.14%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `38.2% <100%> (+3.23%)` | `15 <4> (+4)` | :arrow_up: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `66.66% <71.42%> (-12.41%)` | `6 <5> (-11)` | |
   | 
[...e/livy/server/discovery/LivyDiscoveryManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0xpdnlEaXNjb3ZlcnlNYW5hZ2VyLnNjYWxh)
 | `75% <75%> (ø)` | `4 <4> (?)` | |
   | ... and [12 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...728deb9](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-02 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.17%`.
   > The diff coverage is `82.71%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.54%   -0.18% 
   - Complexity  908  922  +14 
   
 Files   100  103   +3 
 Lines  5674 5709  +35 
 Branches854  853   -1 
   
   + Hits   3899 3913  +14 
   - Misses 1219 1239  +20 
   - Partials556  557   +1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...c/main/java/org/apache/livy/LivyClientBuilder.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-YXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L0xpdnlDbGllbnRCdWlsZGVyLmphdmE=)
 | `83.05% <ø> (ø)` | `17 <0> (ø)` | :arrow_down: |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `75.15% <ø> (+1.24%)` | `20 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.87% <100%> (+0.53%)` | `8 <1> (+1)` | :arrow_up: |
   | 
[...n/java/org/apache/livy/client/http/HttpClient.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-Y2xpZW50LWh0dHAvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2xpdnkvY2xpZW50L2h0dHAvSHR0cENsaWVudC5qYXZh)
 | `86.15% <100%> (+0.66%)` | `16 <1> (+1)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `95.87% <100%> (+0.02%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `38.2% <100%> (+3.23%)` | `15 <4> (+4)` | :arrow_up: |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `66.66% <71.42%> (-12.41%)` | `6 <5> (-11)` | |
   | 
[...e/livy/server/discovery/LivyDiscoveryManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0xpdnlEaXNjb3ZlcnlNYW5hZ2VyLnNjYWxh)
 | `75% <75%> (ø)` | `3 <3> (?)` | |
   | ... and [14 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...104f66b](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment 

[GitHub] [incubator-livy] codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery

2019-08-01 Thread GitBox
codecov-io edited a comment on issue #189: [LIVY-616] Livy Server discovery
URL: https://github.com/apache/incubator-livy/pull/189#issuecomment-517236826
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=h1) 
Report
   > Merging 
[#189](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/1ce266d78fb3475af6154262629ff92f5d342ae6?src=pr=desc)
 will **decrease** coverage by `0.16%`.
   > The diff coverage is `81.57%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/189/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #189  +/-   ##
   
   - Coverage 68.71%   68.54%   -0.17% 
   - Complexity  908  920  +12 
   
 Files   100  103   +3 
 Lines  5674 5704  +30 
 Branches854  853   -1 
   
   + Hits   3899 3910  +11 
   - Misses 1219 1236  +17 
   - Partials556  558   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `78.94% <ø> (-4.39%)` | `1 <0> (ø)` | |
   | 
[.../org/apache/livy/server/discovery/JsonMapper.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0pzb25NYXBwZXIuc2NhbGE=)
 | `100% <100%> (ø)` | `0 <0> (?)` | |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `37.91% <100%> (+2.93%)` | `15 <4> (+4)` | :arrow_up: |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `95.87% <100%> (+0.02%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `66.66% <71.42%> (-12.41%)` | `6 <5> (-11)` | |
   | 
[...pache/livy/server/discovery/DiscoveryManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L0Rpc2NvdmVyeU1hbmFnZXIuc2NhbGE=)
 | `75% <75%> (ø)` | `3 <3> (?)` | |
   | 
[...pache/livy/server/discovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvZGlzY292ZXJ5L1pvb0tlZXBlck1hbmFnZXIuc2NhbGE=)
 | `78.72% <78.72%> (ø)` | `18 <18> (?)` | |
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `73.23% <0%> (-6.34%)` | `33% <0%> (ø)` | |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `50% <0%> (-5.89%)` | `7% <0%> (ø)` | |
   | 
[...in/java/org/apache/livy/rsc/driver/JobWrapper.java](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvSm9iV3JhcHBlci5qYXZh)
 | `82.85% <0%> (-5.72%)` | `8% <0%> (-1%)` | |
   | ... and [7 
more](https://codecov.io/gh/apache/incubator-livy/pull/189/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/189?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/189?src=pr=footer).
 Last update 
[1ce266d...9731aae](https://codecov.io/gh/apache/incubator-livy/pull/189?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).