Skip to content

Fix premature tracer span closure for fire-and-forget and publish#6083

Open
tsegismont wants to merge 2 commits intoeclipse-vertx:masterfrom
tsegismont:issue/6021
Open

Fix premature tracer span closure for fire-and-forget and publish#6083
tsegismont wants to merge 2 commits intoeclipse-vertx:masterfrom
tsegismont:issue/6021

Conversation

@tsegismont
Copy link
Copy Markdown
Member

See #6021

Add MessageConsumerOptions#setAutoProcessed and Message#processed() so consumers can defer trace span closure until async/blocking handler completion.

Some portions of this content were created with the assistance of IBM Bob.

@vietj
Copy link
Copy Markdown
Member

vietj commented Apr 23, 2026

I'd rather name processed by ack which is a common terminology, we also used this for acking MQTT messages

@tsegismont
Copy link
Copy Markdown
Member Author

Done

Comment thread vertx-core/src/main/asciidoc/eventbus.adoc Outdated
Add MessageConsumerOptions#setAutoAck and Message#ack() so consumers can
defer trace span closure until async/blocking handler completion.

Some portions of this content were created with the assistance of IBM Bob.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
@tsegismont tsegismont requested a review from vietj April 29, 2026 12:58
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.

2 participants