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

feat: queue message payloads #20749

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat: queue message payloads #20749

wants to merge 6 commits into from

Conversation

caalador
Copy link
Contributor

Add sent payloads to message
queue and resend if no response
to message inside MaxMessageSuspendTimeout

fixes #20507

Copy link

github-actions bot commented Dec 19, 2024

Test Results

1 160 files  +1  1 160 suites  +1   1h 30m 19s ⏱️ - 1m 18s
7 583 tests +2  7 527 ✅ +3  56 💤 ±0  0 ❌  - 1 
7 923 runs  +1  7 859 ✅  - 2  64 💤 +4  0 ❌  - 1 

Results for commit f49e938. ± Comparison against base commit d17ff83.

♻️ This comment has been updated with latest results.

@caalador caalador force-pushed the issues/20507-client-queue branch 7 times, most recently from 5b7dfd6 to 8cf6097 Compare December 23, 2024 08:56
@caalador caalador force-pushed the issues/20507-client-queue branch 8 times, most recently from e4a6e3e to 0a9ae4d Compare January 8, 2025 12:00
@caalador caalador marked this pull request as ready for review January 8, 2025 12:06
Add sent payloads to message
queue and resend if no response
to message inside MaxMessageSuspendTimeout

fixes #20507
Fix queued message send timing.
@caalador caalador force-pushed the issues/20507-client-queue branch from 0a9ae4d to 7a97b9a Compare January 9, 2025 05:06
@vaadin-bot vaadin-bot added +1.0.0 and removed +0.0.1 labels Jan 10, 2025
@caalador caalador force-pushed the issues/20507-client-queue branch 4 times, most recently from 79f89bc to f7ff95f Compare January 10, 2025 09:07
Fix concurrent issue with custom uidl handler.
@caalador caalador force-pushed the issues/20507-client-queue branch from f7ff95f to 998dfb5 Compare January 10, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client to queue request messages to server and resend them when needed
2 participants