Skip to content

KYLIN-1716 Insight routing error fix#874

Merged
luguosheng1314 merged 1 commit intoapache:masterfrom
Sean-Gu:insight-routing-fix
Oct 16, 2019
Merged

KYLIN-1716 Insight routing error fix#874
luguosheng1314 merged 1 commit intoapache:masterfrom
Sean-Gu:insight-routing-fix

Conversation

@Sean-Gu
Copy link

@Sean-Gu Sean-Gu commented Oct 10, 2019

@asf-ci
Copy link

asf-ci commented Oct 10, 2019

Can one of the admins verify this patch?

1 similar comment
@asf-ci
Copy link

asf-ci commented Oct 10, 2019

Can one of the admins verify this patch?

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #874      +/-   ##
============================================
- Coverage     25.46%   25.46%   -0.01%     
  Complexity     6108     6108              
============================================
  Files          1410     1410              
  Lines         84919    84919              
  Branches      11897    11897              
============================================
- Hits          21626    21624       -2     
- Misses        61215    61217       +2     
  Partials       2078     2078
Impacted Files Coverage Δ Complexity Δ
...org/apache/kylin/rest/util/QueryRequestLimits.java 35.71% <0%> (-4.77%) 5% <0%> (-1%)
...he/kylin/job/impl/threadpool/DefaultScheduler.java 74.41% <0%> (-2.33%) 12% <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 b3ca7e2...2d57583. Read the comment docs.

@luguosheng1314 luguosheng1314 merged commit a6ab91b into apache:master Oct 16, 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.

4 participants