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

fix failing patch updates of search index #1469

Merged

Conversation

dguggemos
Copy link
Contributor

ThingUpdater actor increased its Thing revision after a skipped update which caused subsequent updates to have no effect (revisions of actor and database did not match).

…wise the current revision number is used in the filter of subsequent patch updates instead of the revision number of the last applied update

Signed-off-by: Dominik Guggemos <dominik.guggemos@bosch.io>
Signed-off-by: Dominik Guggemos <dominik.guggemos@bosch.io>
… for subsequent processing of patch update

Signed-off-by: Dominik Guggemos <dominik.guggemos@bosch.io>
@thjaeckle thjaeckle added this to the 3.0.0 milestone Sep 1, 2022
Copy link
Contributor

@yufei-cai yufei-cai left a comment

Choose a reason for hiding this comment

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

LGTM.

@yufei-cai yufei-cai merged commit 451b4e3 into eclipse-ditto:master Sep 2, 2022
@yufei-cai yufei-cai deleted the bugfix/fix-search-patch-updates branch September 2, 2022 13:53
This pull request was closed.
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