Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

linasm
Copy link
Collaborator

@linasm linasm commented Sep 10, 2021

What this PR does / why we need it:
Add AggregateTilesOptions.Process field (enum). This will be more useful than having a boolean.

Special notes for your reviewer:
This overrides #3736.

Does this PR introduce a user-facing and/or backwards incompatible change?:
NONE

Does this PR require updating code package or user-facing documentation?:
NONE

@linasm linasm requested a review from vpranckaitis September 10, 2021 08:57
@codecov
Copy link

codecov bot commented Sep 10, 2021

Codecov Report

Merging #3737 (05c329d) into master (4622985) will decrease coverage by 0.2%.
The diff coverage is 0.0%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #3737     +/-   ##
========================================
- Coverage    57.1%   56.8%   -0.3%     
========================================
  Files         552     552             
  Lines       62937   62950     +13     
========================================
- Hits        35938   35818    -120     
- Misses      23815   23940    +125     
- Partials     3184    3192      +8     
Flag Coverage Δ
aggregator 63.2% <ø> (ø)
cluster ∅ <ø> (∅)
collector 58.4% <ø> (ø)
dbnode 60.5% <0.0%> (-0.3%) ⬇️
m3em 46.4% <ø> (ø)
metrics 19.7% <ø> (ø)
msg 74.4% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 4622985...05c329d. Read the comment docs.

@linasm linasm merged commit 6b5c855 into master Sep 10, 2021
@linasm linasm deleted the linasm/add-AggregateTilesOptions-process branch September 10, 2021 10:51
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.

2 participants