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

[Transaction] Support producer state manager recovery #2

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

Demogorgon314
Copy link
Owner

No description provided.

@Demogorgon314 Demogorgon314 force-pushed the transaction/Implement_ProducerStateManager_recovery branch 5 times, most recently from 49d3dc7 to d393a60 Compare May 30, 2023 00:35
@Demogorgon314 Demogorgon314 changed the title [Transaction] Support producer state manager [Transaction] Support producer state manager recovery May 30, 2023
@Demogorgon314 Demogorgon314 force-pushed the transaction/Implement_ProducerStateManager_recovery branch from d393a60 to f60a822 Compare June 8, 2023 09:27
@Demogorgon314 Demogorgon314 reopened this Jun 9, 2023
@Demogorgon314 Demogorgon314 force-pushed the transaction/Implement_ProducerStateManager_recovery branch 5 times, most recently from 61a3a13 to d7894f4 Compare June 15, 2023 10:29
@Demogorgon314 Demogorgon314 reopened this Jun 15, 2023
@Demogorgon314 Demogorgon314 force-pushed the transaction/Implement_ProducerStateManager_recovery branch from 04a985e to 85570d1 Compare June 21, 2023 00:35
BewareMyPower and others added 13 commits June 27, 2023 12:28
### Motivation

Even if the session timeout is only 1ms, there is still a chance that
the write does not time out.

### Modifications

Run each test in `OffsetTopicWriteTimeoutTest` 10 times and check the
errors could only be the expected error or just `NONE`.
… markers (streamnative#1907)

This PR is cherry-picking from
datastax/starlight-for-kafka@e0636995

### Motivation

Complete the pending request features when the channel is Inactive, or
the exception caught.

Co-authored-by: Enrico Olivelli <[email protected]>
This fixes both completed transactions written to
the transaction log and transaction recovery.
@Demogorgon314 Demogorgon314 force-pushed the transaction/Implement_ProducerStateManager_recovery branch from 2674bcb to 940e90d Compare June 28, 2023 12:18
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