Skip to content

Comments

KYLIN-4242 minor fix#964

Merged
nichunen merged 1 commit intoapache:masterfrom
zhangayqian:KYLIN-4242
Nov 27, 2019
Merged

KYLIN-4242 minor fix#964
nichunen merged 1 commit intoapache:masterfrom
zhangayqian:KYLIN-4242

Conversation

@zhangayqian
Copy link
Contributor

No description provided.

@asf-ci
Copy link

asf-ci commented Nov 27, 2019

Can one of the admins verify this patch?

1 similar comment
@asf-ci
Copy link

asf-ci commented Nov 27, 2019

Can one of the admins verify this patch?

@codecov-io
Copy link

Codecov Report

Merging #964 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #964      +/-   ##
============================================
+ Coverage     25.55%   25.55%   +<.01%     
- Complexity     6138     6139       +1     
============================================
  Files          1412     1412              
  Lines         85137    85137              
  Branches      11935    11935              
============================================
+ Hits          21756    21761       +5     
+ Misses        61272    61269       -3     
+ Partials       2109     2107       -2
Impacted Files Coverage Δ Complexity Δ
...n/rest/security/PasswordPlaceholderConfigurer.java 0% <ø> (ø) 0 <0> (ø) ⬇️
.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java 66.15% <0%> (+2.3%) 0% <0%> (ø) ⬇️
...he/kylin/dict/lookup/cache/RocksDBLookupTable.java 78.37% <0%> (+5.4%) 7% <0%> (+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 f6dfb68...9e448ce. Read the comment docs.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 5294

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.004%) to 28.038%

Totals Coverage Status
Change from base Build 5291: 0.004%
Covered Lines: 23871
Relevant Lines: 85137

💛 - Coveralls

@nichunen nichunen merged commit 716ee8a into apache:master Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants