Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update the runner associated with the workflow #3292

Merged
merged 5 commits into from
Nov 11, 2021
Merged

Conversation

Shinji-IkariG
Copy link
Contributor

What type of PR is this?

  • bug
  • feature
  • enhancement

What does this PR do?

After adding the runner label, it is necessary to distinguish the dedicated runner

Which issue(s)/PR(s) this PR relates to?

x

Special notes for your reviewer, ex. impact of this fix, etc:

On which runners the action task runs

Additional context:

Checklist:

  • Documentation affected (Please add the label if documentation needs to be modified.)
  • Incompatible (If it is incompatible, please describe it and add corresponding label.)
  • Need to cherry-pick (If need to cherry-pick to some branches, please label the destination version(s).)
  • Performance impacted: Consumes more CPU/Memory

Release notes:

Please confirm whether to reflect in release notes and how to describe:

                                                            `

update runner label
update runner label
update runner label
@Shinji-IkariG Shinji-IkariG requested review from HarrisChu and yixinglu and removed request for HarrisChu November 9, 2021 08:27
@yixinglu yixinglu added the ready-for-testing PR: ready for the CI test label Nov 10, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #3292 (5c7b654) into master (58f0b44) will increase coverage by 0.06%.
The diff coverage is 91.19%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3292      +/-   ##
==========================================
+ Coverage   85.24%   85.30%   +0.06%     
==========================================
  Files        1295     1304       +9     
  Lines      118190   119912    +1722     
==========================================
+ Hits       100748   102293    +1545     
- Misses      17442    17619     +177     
Impacted Files Coverage Δ
src/storage/ExprVisitorBase.h 0.00% <0.00%> (ø)
src/storage/exec/StorageIterator.h 94.33% <ø> (+5.45%) ⬆️
src/storage/test/LookupIndexTest.cpp 100.00% <ø> (ø)
src/storage/exec/IndexSelectionNode.h 26.47% <26.47%> (ø)
src/storage/ExprVisitorBase.cpp 29.31% <29.31%> (ø)
src/storage/test/IndexTestUtil.h 66.53% <66.53%> (ø)
src/storage/exec/IndexLimitNode.cpp 66.66% <66.66%> (ø)
src/storage/exec/IndexVertexScanNode.cpp 90.41% <90.41%> (ø)
src/storage/exec/IndexNode.cpp 91.66% <91.66%> (ø)
src/storage/exec/IndexScanNode.cpp 92.50% <92.50%> (ø)
... and 50 more

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 1a4c4bd...5c7b654. Read the comment docs.

@yixinglu yixinglu merged commit 4f832fb into master Nov 11, 2021
@yixinglu yixinglu deleted the Shinji-IkariG-patch-1 branch November 11, 2021 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants