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
SyncEngine: Fix detection of backup or full resync #5104
Conversation
|
Does that go into 2.2.3 ? |
@@ -681,6 +681,10 @@ void SyncEngine::startSync() | |||
s_anySyncRunning = true; | |||
_syncRunning = true; | |||
_anotherSyncNeeded = false; | |||
_hasNoneFiles = false; | |||
_hasRemoveFile = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those two are already reset in slotDiscoveryJobFinished
If you think this should go in 2.2.3, we should make sure that we give some time for the RC to be used by people, and we should do extra QA for it. We shouldn't just put land the patch and release the day after I think. |
OK let's have it in 2.2.3 and make a rc tomorrow.. |
Once upon a time, the SyncEngine was instantiated once per sync. But now that the SyncEngine is kept between sync, we need to reset all these variable between syncs.
e13f2f7
to
5dc3d55
Compare
👍 |
👍 from me. |
@davitol FYI nightlies should have it |
Once upon a time, the SyncEngine was instantiated once per sync. But now that the SyncEngine is kept between sync, we need to reset all these variable between syncs.