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(mvux): Properties update might be deferred too long if dispatcher is under heavy stress #2250

Merged
merged 2 commits into from
Apr 12, 2024

Conversation

dr1rrb
Copy link
Member

@dr1rrb dr1rrb commented Apr 11, 2024

fixes #2188

Bugfix

Properties update might be deferred too long if dispatcher is under heavy stress

What is the current behavior?

When a property change often, we wait for the dispatcher to have process the last update before fetching the next update from the bg thread.

What is the new behavior?

Updates are scheduled as soon as possible from the bg thread and de-duplicated if

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
  • Contains NO breaking changes
  • Updated the Release Notes
  • Associated with an issue (GitHub or internal)

@dr1rrb dr1rrb force-pushed the dev/dr/mvuxPropChangedJamming branch from dfbfe44 to 042f44f Compare April 12, 2024 11:45
@dr1rrb dr1rrb force-pushed the dev/dr/mvuxPropChangedJamming branch from 042f44f to 5272fa7 Compare April 12, 2024 13:41
@dr1rrb dr1rrb enabled auto-merge April 12, 2024 13:57
@dr1rrb dr1rrb merged commit 0c9d21a into main Apr 12, 2024
15 of 16 checks passed
@dr1rrb dr1rrb deleted the dev/dr/mvuxPropChangedJamming branch April 12, 2024 14:13
@agneszitte
Copy link
Contributor

@Mergifyio backport release/stable/4.1

Copy link

mergify bot commented Apr 23, 2024

backport release/stable/4.1

✅ Backports have been created

dr1rrb added a commit that referenced this pull request Apr 23, 2024
….1/pr-2250

fix(mvux): Properties update might be deferred too long if dispatcher is under heavy stress (backport #2250)
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.

[MVUX] Binding issue
3 participants