Skip to content

cds 9: task queue#1790

Merged
renejeglinsky merged 71 commits into
mainfrom
task-queue
May 28, 2025
Merged

cds 9: task queue#1790
renejeglinsky merged 71 commits into
mainfrom
task-queue

Conversation

@David-Kunz
Copy link
Copy Markdown
Contributor

Only for cds 9

@David-Kunz David-Kunz requested a review from smahati as a code owner April 24, 2025 09:53
Comment thread node.js/queue.md Outdated
@David-Kunz David-Kunz changed the title task queue cds 9: task queue Apr 24, 2025
@David-Kunz David-Kunz requested a review from sjvans April 30, 2025 08:39
Comment thread node.js/_menu.md
Comment thread node.js/messaging.md
Comment thread node.js/queue.md
Comment thread node.js/queue.md Outdated
- `storeLastError` (default `true`): Specifies if error information of the last failed emit should be stored in the tasks table.
- `parallel` (default `true`): Specifies if messages are sent in parallel (faster but the order isn't guaranteed).
- `timeout` (default `"1h"`): The time after which a message with `status = "processing"` can be processed again. Only for `legacyLocking = false`.
- `legacyLocking` (default `true`): If set to `false`, database locks are only used to set the status of the message to `processing` to prevent long-kept database locks. This is recommended but incompatible for parallel usage with `@sap/cds^8` instances.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What is recommended? The default or setting it to false?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

setting it to false. however, can only be done if all task runners do so (prerequisite: they all must be on cds9)

Comment thread node.js/queue.md Outdated
Comment thread node.js/queue.md Outdated
@sjvans
Copy link
Copy Markdown
Contributor

sjvans commented May 26, 2025

my review: #1878

Co-authored-by: Dr. David A. Kunz <david.kunz@sap.com>
github-actions[bot]

This comment was marked as resolved.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions[bot]

This comment was marked as resolved.

@sjvans
Copy link
Copy Markdown
Contributor

sjvans commented May 27, 2025

@renejeglinsky @smahati ready from our side

@renejeglinsky renejeglinsky merged commit 87b3dad into main May 28, 2025
4 of 5 checks passed
@renejeglinsky renejeglinsky deleted the task-queue branch May 28, 2025 07:02
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.

4 participants