Skip to content

Checkpoint restore precendence cleanup#1512

Merged
prateekm merged 15 commits intoapache:state-backend-async-commitfrom
dxichen:checkpoint-restore-precendence-cleanup
Aug 5, 2021
Merged

Checkpoint restore precendence cleanup#1512
prateekm merged 15 commits intoapache:state-backend-async-commitfrom
dxichen:checkpoint-restore-precendence-cleanup

Conversation

@dxichen
Copy link
Member

@dxichen dxichen commented Aug 3, 2021

Merging internal changes to oss:

@dxichen 
SnapshotIndexSerde serialization fixes …
354620f
@dxichen
Checkpoint-tool changes for checkpoint v2 …
48a8785
@dxichen
HOT FIX: KafkaConsumerProxy without any registered TopicPartitions
2063155
@dxichen
Fix async cleanup TaskMetric …
271ef64
@shekhars-li
@dxichen
Added lastCommitNs and commitTimedOut metrics, updated default timeou… …
7d61a7d
@shekhars-li
@dxichen
Updated log line in BlobStoreUtil
7f5ff3a
@dxichen
Checkpoint version config precedence list …
35c0dbc
@dxichen
Per store restore manager config precendence …
7676da9
@dxichen
Job level state backend configs …
5379f97
@dxichen
Fixed case where there are no stores to backup for CSM
0697457
@dxichen
Added tests for CSM for non configured stores …
e74c241
@dxichen
Cleanup of TODOs
@shekhars-li 
Test fix for failing job: only backup persistent stores

dxichen and others added 15 commits August 2, 2021 20:34
RB=2624444

RB=2624448
G=samza-reviewers
R=shesharm
A=shesharm,bkumaras
RB=2621726
G=samza-reviewers
A=pmaheshw
RB=2632353
G=samza-reviewers
R=pmaheshw,shesharm
A=pmaheshw
…t to 30 mins

RB=2634862

RB=2634867
G=samza-reviewers
R=pmaheshw,dchen1
A=dchen1
RB=2652787

RB=2652789
G=samza-reviewers
R=pmaheshw,shesharm
A=pmaheshw
RB=2675917
G=samza-reviewers
R=pmaheshw,shesharm
A=pmaheshw,shesharm
RB=2721143
G=samza-reviewers
R=shesharm
A=
@prateekm prateekm merged commit 48699c4 into apache:state-backend-async-commit Aug 5, 2021
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.

3 participants

Comments