Skip to content

KYLIN-4308 Make kylin.sh tips clearer and more explicit#1011

Merged
nichunen merged 1 commit intoapache:masterfrom
weibin0516:KYLIN-4308
Dec 20, 2019
Merged

KYLIN-4308 Make kylin.sh tips clearer and more explicit#1011
nichunen merged 1 commit intoapache:masterfrom
weibin0516:KYLIN-4308

Conversation

@weibin0516
Copy link

No description provided.

@asf-ci
Copy link

asf-ci commented Dec 20, 2019

Can one of the admins verify this patch?

1 similar comment
@asf-ci
Copy link

asf-ci commented Dec 20, 2019

Can one of the admins verify this patch?

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff             @@
##             master   #1011      +/-   ##
===========================================
+ Coverage      25.5%   25.5%   +<.01%     
  Complexity     6177    6177              
===========================================
  Files          1416    1416              
  Lines         85967   85967              
  Branches      12059   12059              
===========================================
+ Hits          21927   21929       +2     
+ Misses        61902   61900       -2     
  Partials       2138    2138
Impacted Files Coverage Δ Complexity Δ
...he/kylin/job/impl/threadpool/DefaultScheduler.java 76.74% <0%> (+2.32%) 12% <0%> (ø) ⬇️

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 bb839ff...0098cfd. Read the comment docs.

Copy link

@nichunen nichunen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nichunen nichunen merged commit 1c817c3 into apache:master Dec 20, 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

Comments