You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@livy.apache.org by "askhatri (via GitHub)" <gi...@apache.org> on 2023/08/17 07:48:46 UTC
[GitHub] [incubator-livy] askhatri opened a new pull request, #412: [LIVY-986][SERVER] Adding null pointer check for SessionInfo
askhatri opened a new pull request, #412:
URL: https://github.com/apache/incubator-livy/pull/412
## What changes were proposed in this pull request?
[LIVY-986][SERVER] Adding null pointer check for SessionInfo
## What changes were proposed in this pull request?
Adding null check for SessionInfo parameters "conf", "archives", "jars", "pyFiles" and "files"
JIRA: https://issues.apache.org/jira/browse/LIVY-986
## How was this patch tested?
Tested manually by running session recovery.
Please review https://livy.incubator.apache.org/community/ before opening a pull request.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-livy] codecov-commenter commented on pull request #412: [LIVY-986][SERVER] Adding null pointer check for SessionInfo
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #412:
URL: https://github.com/apache/incubator-livy/pull/412#issuecomment-1681878284
## [Codecov](https://app.codecov.io/gh/apache/incubator-livy/pull/412?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
> Merging [#412](https://app.codecov.io/gh/apache/incubator-livy/pull/412?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (b2d0dc7) into [master](https://app.codecov.io/gh/apache/incubator-livy/commit/b175ba1bd1210e2d1db61bdf19a3dbfc1374dd8c?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (b175ba1) will **decrease** coverage by `37.04%`.
> Report is 1 commits behind head on master.
> The diff coverage is `44.44%`.
```diff
@@ Coverage Diff @@
## master #412 +/- ##
=============================================
- Coverage 65.61% 28.58% -37.04%
+ Complexity 952 379 -573
=============================================
Files 103 103
Lines 6044 6059 +15
Branches 911 916 +5
=============================================
- Hits 3966 1732 -2234
- Misses 1534 3973 +2439
+ Partials 544 354 -190
```
| [Files Changed](https://app.codecov.io/gh/apache/incubator-livy/pull/412?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...server/interactive/InteractiveSessionServlet.scala](https://app.codecov.io/gh/apache/incubator-livy/pull/412?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zZXJ2ZXIvaW50ZXJhY3RpdmUvSW50ZXJhY3RpdmVTZXNzaW9uU2VydmxldC5zY2FsYQ==) | `56.12% <44.44%> (-6.02%)` | :arrow_down: |
... and [85 files with indirect coverage changes](https://app.codecov.io/gh/apache/incubator-livy/pull/412/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
:mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-livy] gyogal merged pull request #412: [LIVY-986][SERVER] Adding null pointer check for SessionInfo
Posted by "gyogal (via GitHub)" <gi...@apache.org>.
gyogal merged PR #412:
URL: https://github.com/apache/incubator-livy/pull/412
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org