Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Provided a better solution for reading dead outbox entries #1940

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.

Sign up for GitHub

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

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

Provided a better solution for reading dead outbox entries #1940

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ed8c785
Provided a better solution for reading dead outbox entries
t-bonk Jun 27, 2025
af2aa4a
Merge branch 'main' into java-outbox-dlq-details
t-bonk Jul 4, 2025
a7b6202
Merge branch 'main' into java-outbox-dlq-details
renejeglinsky Jul 4, 2025
e01d281
Added a before handler to modify the select statement
t-bonk Jul 14, 2025
ccd12db
Merge branch 'main' into java-outbox-dlq-details
t-bonk Jul 14, 2025
5ac85af
Apply suggestions from code review
t-bonk Jul 18, 2025
37e31e0
Merge branch 'main' into java-outbox-dlq-details
t-bonk Jul 18, 2025
0d5b5ed
changes after code review
t-bonk Jul 18, 2025
589ee70
Update java/outbox.md
t-bonk Aug 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Apply suggestions from code review 
Co-authored-by: BraunMatthias <[email protected]>
  • Loading branch information
@t-bonk @BraunMatthias
t-bonk and BraunMatthias authored Jul 18, 2025
commit 5ac85af2f2d8cac3e15672bf0e43eec392037b2a
10 changes: 4 additions & 6 deletions java/outbox.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -357,8 +357,7 @@ It is crucial to make the service `OutboxDeadLetterQueueService` accessible for

### Reading Dead Entries

This filtering of dead entries is done on the database by adding `where` clauses for each outbox and their maximum number of retries. The following code provides the handler for the `DeadLetterQueueService` that modifies the where clause by adding
additional conditions for filtering the outbox entries:
Filtering the dead entries is done by adding by adding an appropriate `where`-clause to `READ`-queries on the outbox message entries which reached maximum number of retries. The following code provides an example handler implementation defining this behaviour for the `DeadLetterQueueService`:

```java
@Component
Expand All @@ -372,11 +371,10 @@ public class DeadOutboxMessagesHandler implements EventHandler {
}

@Before(entity = DeadOutboxMessages_.CDS_NAME)
public void modifyWhereClause(CdsReadEventContext context) {
CqnSelect cqn = context.getCqn();
public void addDeadEntryFilter(CdsReadEventContext context) {
Optional<Predicate> outboxFilters = this.createOutboxFilters(context.getCdsRuntime());
CqnSelect modifiedCqn = copy(
cqn,
context.getCqn(),
new Modifier() {
@Override
public CqnPredicate where(Predicate where) {
Expand Down Expand Up @@ -455,7 +453,7 @@ The injected `PersistenceService` instance is used to perform the operations on
[Learn more about CQL statement inspection.](./working-with-cql/query-introspection#cqnanalyzer){.learn-more}

::: tip Use paging logic
Avoid to read all entries of the `cds.outbox.Messages` or `OutboxDeadLetterQueueService.DeadOutboxMessages` table at once, as the size of an entry is unpredictable and depends on the size of the payload. Prefer paging logic instead.
Avoid to read all outbox entries at once as a single entry can have significant size reflecting the request's payload. Prefer `READ`-queries with paging instead.
:::

## Observability using Open Telemetry
Expand Down
Loading