Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[KYUUBI-118]fix npe bug with incremental result collection #119

Merged
merged 2 commits into from
Nov 7, 2018

Conversation

yaooqinn
Copy link
Member

@yaooqinn yaooqinn commented Nov 7, 2018

No description provided.

@yaooqinn yaooqinn added this to the v0.4.0 milestone Nov 7, 2018
@codecov
Copy link

codecov bot commented Nov 7, 2018

Codecov Report

Merging #119 into master will increase coverage by 0.31%.
The diff coverage is 92.1%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #119      +/-   ##
============================================
+ Coverage     72.24%   72.56%   +0.31%     
- Complexity      529      542      +13     
============================================
  Files            72       72              
  Lines          3423     3266     -157     
  Branches        491      475      -16     
============================================
- Hits           2473     2370     -103     
+ Misses          676      639      -37     
+ Partials        274      257      -17
Impacted Files Coverage Δ Complexity Δ
...ver/src/main/scala/org/apache/spark/SparkEnv.scala 76.92% <ø> (+8.17%) 0 <0> (ø) ⬇️
...la/yaooqinn/kyuubi/spark/SparkSessionWithUGI.scala 75% <100%> (+0.24%) 17 <9> (+5) ⬆️
...scala/yaooqinn/kyuubi/session/SessionManager.scala 77.27% <100%> (ø) 28 <0> (ø) ⬇️
...a/yaooqinn/kyuubi/operation/OperationManager.scala 79.03% <100%> (ø) 28 <0> (ø) ⬇️
...la/yaooqinn/kyuubi/operation/KyuubiOperation.scala 55.91% <50%> (ø) 39 <0> (ø) ⬇️
.../scala/yaooqinn/kyuubi/session/KyuubiSession.scala 72.13% <90.9%> (+0.23%) 44 <10> (+8) ⬆️
...ain/scala/yaooqinn/kyuubi/KyuubiSQLException.scala 95.45% <93.75%> (+0.85%) 8 <1> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34af4d0...5a957ef. Read the comment docs.

@yaooqinn yaooqinn merged commit d1d34e5 into master Nov 7, 2018
@yaooqinn yaooqinn deleted the KYUUBI-118 branch November 7, 2018 09:37
@yaooqinn yaooqinn modified the milestone: v0.4.0 Dec 21, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant