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

[aggregator] Update flush times after each flush #3890

Merged
merged 30 commits into from
Nov 6, 2021
Merged
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
d16d7a5
[aggregator] Update flush times after each flush
arnikola Oct 29, 2021
d5e2dd8
Merge branch 'master' into arnikola/update-flush
arnikola Nov 4, 2021
b92ad82
Fix up tests, deprecate the flushTimesPersistEvery option
arnikola Nov 4, 2021
3eb513d
Update tests
arnikola Nov 4, 2021
50e6859
Excise remaining FlushTimesPersistEvery
arnikola Nov 4, 2021
24b9e81
Fix tests, lint
arnikola Nov 4, 2021
b0caeaf
Filtering through eager shutdown flag for back-compat
arnikola Nov 4, 2021
4c02198
Add timeout metric for alerting
arnikola Nov 4, 2021
8694a72
lint
arnikola Nov 4, 2021
3e2040b
Merge branch 'master' into arnikola/update-flush
arnikola Nov 4, 2021
c3b92b7
Always eager close aggregator
arnikola Nov 4, 2021
d9e5191
Merge branch 'arnikola/update-flush' of github.com:m3db/m3 into arnik…
arnikola Nov 4, 2021
6537fa7
Remove logging
arnikola Nov 4, 2021
7b42594
Re-add FlushTimesPersistEvery option
arnikola Nov 5, 2021
284a47c
PR response
arnikola Nov 5, 2021
932a5ca
Do not have a default defaultFlushTimesPersistEvery, to match config
arnikola Nov 5, 2021
ec4100f
Touchups for clarity/saner defaults
arnikola Nov 5, 2021
6164727
Re-add param (oops)
arnikola Nov 5, 2021
5040f33
responsible
arnikola Nov 5, 2021
cf0c3bb
Merge branch 'master' into arnikola/update-flush
arnikola Nov 5, 2021
aefd482
Close
arnikola Nov 5, 2021
40346ba
Merge branch 'arnikola/update-flush' of github.com:m3db/m3 into arnik…
arnikola Nov 5, 2021
c282209
Merge branch 'master' into arnikola/update-flush
arnikola Nov 5, 2021
9ba8ff0
Merge branch 'arnikola/update-flush' of github.com:m3db/m3 into arnik…
arnikola Nov 5, 2021
6152b61
response
arnikola Nov 5, 2021
666503f
wip
arnikola Nov 5, 2021
a29e989
Test cleanup
arnikola Nov 5, 2021
278d7d2
Response
arnikola Nov 5, 2021
8b480d3
Merge branch 'master' into arnikola/update-flush
arnikola Nov 5, 2021
86dc55d
Merge branch 'master' into arnikola/update-flush
arnikola Nov 6, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
responsible
arnikola committed Nov 5, 2021
commit 5040f33f5c3489c7e305108e4c5557dc4fab2149
2 changes: 1 addition & 1 deletion src/aggregator/aggregator/aggregator.go
Original file line number Diff line number Diff line change
@@ -177,7 +177,7 @@ func (agg *aggregator) Open() error {

// NB: placement tick watches the placement manager, and initializes a
// topology change if the placement is updated. This changes which shards this
// aggregator is repsonsible for, and initiates leader elections. In the
// aggregator is responsible for, and initiates leader elections. In the
// scenario where a placement change is received when this aggregator is
// closed, it's fine to ignore the result of the placement update, as applying
// the change only affects the current aggregator that is being closed anyway.