Skip to content

lastSequenceNr should reflect the snapshot sequence - #27993

Merged
patriknw merged 1 commit into
masterfrom
wip-lastSequenceNr-patriknw
Oct 14, 2019
Merged

patriknw merged 1 commit into
masterfrom
wip-lastSequenceNr-patriknw

Conversation

@patriknw

Copy link
Copy Markdown
Contributor

Forward port of #27496

(cherry picked from commit df2a1d8)

…h 0 when journal is empty (#27496)

* lastSequenceNr should reflect the snapshot sequence number and not start
at 0 when there was no journal to begin with.

* Use LevelDBJournal in Test Case

The test case now properly simulates a recovery from a snapshot without
a journal.

(cherry picked from commit df2a1d8)
@patriknw patriknw added this to the 2.6.0-RC1 milestone Oct 14, 2019
@akka-ci akka-ci added validating PR is currently being validated by Jenkins tested PR that was successfully built and tested by Jenkins and removed validating PR is currently being validated by Jenkins labels Oct 14, 2019
@akka-ci

akka-ci commented Oct 14, 2019

Copy link
Copy Markdown

Test PASSed.

@patriknw

Copy link
Copy Markdown
Contributor Author

merging since it's a cherry pick from other PR that has been reviewed, I'll follow up with the additional things mention in other pr

@patriknw
patriknw merged commit 6557967 into master Oct 14, 2019
@patriknw
patriknw deleted the wip-lastSequenceNr-patriknw branch October 14, 2019 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport tested PR that was successfully built and tested by Jenkins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants