Skip to content
New issue

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

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

Already on GitHub? # to your account

[ISSUE#266] Support read state from statemachine with RaftLog-Read mode #266 #267

Conversation

TheR1sing3un
Copy link
Contributor

issue: #266

TheR1sing3un and others added 17 commits October 12, 2022 16:32
1. fix wrong begin index after applying snapshot
1. fix wrong recover after reset offset
1. pass checkstyle
1. support read from statemachine with raft-log read mode
2. support user define request processor in server and client
2. refactor the
appending entry logic
3. refactor some code for better logical
relationship
1. add more tests in AppendAndReadTest
[ISSUE 245] fix wrong recover logic after reseting offset
1. fix some problems in snapshot mode
1. fix some problems in snapshot mode
1. use generic to refactor some methods
1. ignore flaky test temporarily
…n_index_after_snapshot

[ISSUE#268]Fix wrong ledgerIndex and wrong generated snapshot in snapshot mode
@tsunghanjacktsai tsunghanjacktsai self-requested a review February 7, 2023 01:45
Copy link
Contributor

@tsunghanjacktsai tsunghanjacktsai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, remember to resolve the conflicts before merging.

1. support read from statemachine with raft-log read mode
2. support user define request processor in server and client
2. refactor the
appending entry logic
3. refactor some code for better logical
relationship
1. add more tests in AppendAndReadTest
1. use generic to refactor some methods
1. ignore flaky test temporarily
…to support_linearizability_read

# Conflicts:
#	dledger/src/main/java/io/openmessaging/storage/dledger/DLedgerEntryPusher.java
@TheR1sing3un
Copy link
Contributor Author

LGTM, remember to resolve the conflicts before merging.

Done~ Plz merge it. Thks!

@tsunghanjacktsai tsunghanjacktsai merged commit 5c09ea6 into openmessaging:master Feb 9, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants