[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-13 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.17%`.
   > The diff coverage is `80%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.31%   +0.17% 
   - Complexity  959  961   +2 
   
 Files   104  105   +1 
 Lines  5946 6000  +54 
 Branches899  902   +3 
   
   + Hits   4052 4099  +47 
   - Misses 1312 1317   +5 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.93% <66.66%> (+1.45%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `89.65% <89.65%> (ø)` | `2 <2> (?)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | ... and [3 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...04cf38a](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-13 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.17%`.
   > The diff coverage is `80%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.31%   +0.17% 
   - Complexity  959  961   +2 
   
 Files   104  105   +1 
 Lines  5946 6000  +54 
 Branches899  902   +3 
   
   + Hits   4052 4099  +47 
   - Misses 1312 1317   +5 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.93% <66.66%> (+1.45%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `89.65% <89.65%> (ø)` | `2 <2> (?)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | ... and [3 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...04cf38a](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-13 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.17%`.
   > The diff coverage is `80%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.31%   +0.17% 
   - Complexity  959  961   +2 
   
 Files   104  105   +1 
 Lines  5946 6000  +54 
 Branches899  902   +3 
   
   + Hits   4052 4099  +47 
   - Misses 1312 1317   +5 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.93% <66.66%> (+1.45%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `89.65% <89.65%> (ø)` | `2 <2> (?)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | ... and [3 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...04cf38a](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-11 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.14%`.
   > The diff coverage is `80.95%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.29%   +0.14% 
   - Complexity  959  962   +3 
   
 Files   104  105   +1 
 Lines  5946 5999  +53 
 Branches899  902   +3 
   
   + Hits   4052 4097  +45 
   - Misses 1312 1318   +6 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.06% <66.66%> (+0.57%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `92.85% <92.85%> (ø)` | `2 <2> (?)` | |
   | ... and [4 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...b946126](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-11 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.16%`.
   > The diff coverage is `80.95%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.31%   +0.16% 
   - Complexity  959  962   +3 
   
 Files   104  105   +1 
 Lines  5946 5999  +53 
 Branches899  902   +3 
   
   + Hits   4052 4098  +46 
   - Misses 1312 1317   +5 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.49% <66.66%> (+1.01%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `92.85% <92.85%> (ø)` | `2 <2> (?)` | |
   | ... and [4 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...b946126](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-11 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.16%`.
   > The diff coverage is `80.95%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.31%   +0.16% 
   - Complexity  959  962   +3 
   
 Files   104  105   +1 
 Lines  5946 5999  +53 
 Branches899  902   +3 
   
   + Hits   4052 4098  +46 
   - Misses 1312 1317   +5 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.49% <66.66%> (+1.01%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `92.85% <92.85%> (ø)` | `2 <2> (?)` | |
   | ... and [4 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...b946126](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-10 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.17%`.
   > The diff coverage is `81.92%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.32%   +0.17% 
   - Complexity  959  961   +2 
   
 Files   104  105   +1 
 Lines  5946 5998  +52 
 Branches899  902   +3 
   
   + Hits   4052 4098  +46 
   - Misses 1312 1316   +4 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.93% <66.66%> (+1.45%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `96.29% <96.29%> (ø)` | `2 <2> (?)` | |
   | ... and [3 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...e4ebea1](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-10 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **increase** coverage by `0.17%`.
   > The diff coverage is `81.92%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   + Coverage 68.14%   68.32%   +0.17% 
   - Complexity  959  961   +2 
   
 Files   104  105   +1 
 Lines  5946 5998  +52 
 Branches899  902   +3 
   
   + Hits   4052 4098  +46 
   - Misses 1312 1316   +4 
   - Partials582  584   +2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `65.11% <0%> (-1.56%)` | `17 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.2% <100%> (+0.07%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `73.33% <50%> (+0.91%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `34.93% <66.66%> (+1.45%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `81.48% <78.94%> (-0.34%)` | `30 <3> (+3)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `77.27% <91.66%> (-2.73%)` | `6 <5> (-4)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `96.29% <96.29%> (ø)` | `2 <2> (?)` | |
   | ... and [3 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...e4ebea1](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-04 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **decrease** coverage by `39.86%`.
   > The diff coverage is `50%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@  Coverage Diff  @@
   ## master #274   +/-   ##
   =
   - Coverage 68.14%   28.27%   -39.87% 
   + Complexity  959  357  -602 
   =
 Files   104  105+1 
 Lines  5946 6001   +55 
 Branches899  903+4 
   =
   - Hits   4052 1697 -2355 
   - Misses 1312 3960 +2648 
   + Partials582  344  -238
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `56% <0%> (-5.23%)` | `9 <0> (-2)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `0% <0%> (-66.67%)` | `0 <0> (-17)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `0% <0%> (-100%)` | `0 <0> (-5)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `0% <0%> (-100%)` | `0 <0> (-8)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `95.26% <100%> (-0.88%)` | `18 <0> (-3)` | |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `32.45% <25%> (-1.03%)` | `11 <0> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `40.74% <40.74%> (ø)` | `2 <2> (?)` | |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `63.63% <57.14%> (-18.19%)` | `20 <2> (-7)` | |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `56.25% <66.66%> (-16.17%)` | `1 <0> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `68.18% <83.33%> (-11.82%)` | `6 <5> (-4)` | |
   | ... and [81 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...27e6acb](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).
 Read the [comment

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-02-04 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **decrease** coverage by `39.86%`.
   > The diff coverage is `50%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@  Coverage Diff  @@
   ## master #274   +/-   ##
   =
   - Coverage 68.14%   28.27%   -39.87% 
   + Complexity  959  357  -602 
   =
 Files   104  105+1 
 Lines  5946 6001   +55 
 Branches899  903+4 
   =
   - Hits   4052 1697 -2355 
   - Misses 1312 3960 +2648 
   + Partials582  344  -238
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `56% <0%> (-5.23%)` | `9 <0> (-2)` | |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `0% <0%> (-66.67%)` | `0 <0> (-17)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `0% <0%> (-100%)` | `0 <0> (-5)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `0% <0%> (-100%)` | `0 <0> (-8)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `95.26% <100%> (-0.88%)` | `18 <0> (-3)` | |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `32.45% <25%> (-1.03%)` | `11 <0> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `40.74% <40.74%> (ø)` | `2 <2> (?)` | |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `63.63% <57.14%> (-18.19%)` | `20 <2> (-7)` | |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `56.25% <66.66%> (-16.17%)` | `1 <0> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `68.18% <83.33%> (-11.82%)` | `6 <5> (-4)` | |
   | ... and [81 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...27e6acb](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).
 Read the [comment

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-01-15 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **decrease** coverage by `0.08%`.
   > The diff coverage is `61.33%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   - Coverage 68.14%   68.06%   -0.09% 
   - Complexity  959  962   +3 
   
 Files   104  105   +1 
 Lines  5946 5999  +53 
 Branches899  902   +3 
   
   + Hits   4052 4083  +31 
   - Misses 1312 1331  +19 
   - Partials582  585   +3
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `76.19% <0%> (-3.81%)` | `10 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.17% <100%> (+0.03%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `64.58% <42.85%> (-2.09%)` | `17 <0> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `50% <50%> (ø)` | `2 <2> (?)` | |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `75% <66.66%> (+2.58%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `35.06% <72.72%> (+1.58%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `80.55% <73.68%> (-1.27%)` | `27 <2> (ø)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | ... and [5 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...caf8ac0](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).
 Read

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-01-15 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **decrease** coverage by `0.08%`.
   > The diff coverage is `61.33%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   - Coverage 68.14%   68.06%   -0.09% 
   - Complexity  959  962   +3 
   
 Files   104  105   +1 
 Lines  5946 5999  +53 
 Branches899  902   +3 
   
   + Hits   4052 4083  +31 
   - Misses 1312 1331  +19 
   - Partials582  585   +3
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `76.19% <0%> (-3.81%)` | `10 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.17% <100%> (+0.03%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `64.58% <42.85%> (-2.09%)` | `17 <0> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `50% <50%> (ø)` | `2 <2> (?)` | |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `75% <66.66%> (+2.58%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `35.06% <72.72%> (+1.58%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `80.55% <73.68%> (-1.27%)` | `27 <2> (ø)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | ... and [5 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...caf8ac0](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).
 Read

[GitHub] [incubator-livy] codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed session id generation

2020-01-15 Thread GitBox 
codecov-io edited a comment on issue #274: [LIVY-721][SERVER] Distributed 
session id generation
URL: https://github.com/apache/incubator-livy/pull/274#issuecomment-572834042
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=h1) 
Report
   > Merging 
[#274](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/66b5833e413bc10e39e3b92b585f496444c147d4?src=pr=desc)
 will **decrease** coverage by `0.08%`.
   > The diff coverage is `61.33%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/274/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #274  +/-   ##
   
   - Coverage 68.14%   68.06%   -0.09% 
   - Complexity  959  962   +3 
   
 Files   104  105   +1 
 Lines  5946 5999  +53 
 Branches899  902   +3 
   
   + Hits   4052 4083  +31 
   - Misses 1312 1331  +19 
   - Partials582  585   +3
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...he/livy/server/recovery/FileSystemStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvRmlsZVN5c3RlbVN0YXRlU3RvcmUuc2NhbGE=)
 | `60% <0%> (-1.23%)` | `11 <0> (ø)` | |
   | 
[...org/apache/livy/server/recovery/SessionStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU2Vzc2lvblN0b3JlLnNjYWxh)
 | `76.19% <0%> (-3.81%)` | `10 <0> (ø)` | |
   | 
[...che/livy/server/recovery/ZooKeeperStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyU3RhdGVTdG9yZS5zY2FsYQ==)
 | `90% <0%> (-10%)` | `8 <0> (ø)` | |
   | 
[...rver/src/main/scala/org/apache/livy/LivyConf.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9MaXZ5Q29uZi5zY2FsYQ==)
 | `96.17% <100%> (+0.03%)` | `21 <0> (ø)` | :arrow_down: |
   | 
[...apache/livy/server/recovery/ZooKeeperManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvWm9vS2VlcGVyTWFuYWdlci5zY2FsYQ==)
 | `64.58% <42.85%> (-2.09%)` | `17 <0> (ø)` | |
   | 
[.../org/apache/livy/sessions/SessionIdGenerator.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uSWRHZW5lcmF0b3Iuc2NhbGE=)
 | `50% <50%> (ø)` | `2 <2> (?)` | |
   | 
[...a/org/apache/livy/server/recovery/StateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvU3RhdGVTdG9yZS5zY2FsYQ==)
 | `75% <66.66%> (+2.58%)` | `1 <0> (ø)` | :arrow_down: |
   | 
[...main/scala/org/apache/livy/server/LivyServer.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvTGl2eVNlcnZlci5zY2FsYQ==)
 | `35.06% <72.72%> (+1.58%)` | `11 <0> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/sessions/SessionManager.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXNzaW9ucy9TZXNzaW9uTWFuYWdlci5zY2FsYQ==)
 | `80.55% <73.68%> (-1.27%)` | `27 <2> (ø)` | |
   | 
[...che/livy/server/recovery/BlackholeStateStore.scala](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvcmVjb3ZlcnkvQmxhY2tob2xlU3RhdGVTdG9yZS5zY2FsYQ==)
 | `83.33% <80%> (-16.67%)` | `5 <4> (ø)` | |
   | ... and [5 
more](https://codecov.io/gh/apache/incubator-livy/pull/274/diff?src=pr=tree-more)
 | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/274?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/274?src=pr=footer).
 Last update 
[66b5833...caf8ac0](https://codecov.io/gh/apache/incubator-livy/pull/274?src=pr=lastupdated).
 Read