Skip to content

Make EventBus Timeout Configurable#33

Merged
sbesson merged 1 commit intoglencoesoftware:masterfrom
kkoz:configure-timeout
Mar 5, 2026
Merged

Make EventBus Timeout Configurable#33
sbesson merged 1 commit intoglencoesoftware:masterfrom
kkoz:configure-timeout

Conversation

@kkoz
Copy link
Copy Markdown
Member

@kkoz kkoz commented Mar 3, 2026

Fixes #32

See glencoesoftware/omero-ms-image-region#160

This PR is mostly for consistency with omero-ms-image-region in allowing the configuration of the EventBus send timeout in the config.yml file.

Testing

Import a test image like 'test&sizeX=2304&sizeY=2304&sizeZ=12&sizeC=3&pixelType=uint16&sleepOpenBytes=1000.fake'. Note the image ID. Set the timeout to be less than 1000 ms and call

/tile/<image ID>/0/0/0?x=512&y=512&w=480&h=360&format=png

You should induce the timeout.
Increase the timeout in the config to be > 1000 ms and the request should succeed.

@kkoz kkoz requested a review from sbesson March 3, 2026 19:33
Copy link
Copy Markdown
Member

@sbesson sbesson left a comment

Choose a reason for hiding this comment

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

Similarly to glencoesoftware/omero-ms-thumbnail#39 (review), used a synthetic file introducing a 5s delay when fetching a plane/tile.

When the micro-service is configured with a timeout of 5000 or lower, requests to the tile/ endpoint with this image fail with a io.vertx.core.eventbus.ReplyException: Timed out after waiting 5000(ms) for a reply as expection. Increasing the timeout via configuration file is sufficient to address the timeout issues

@sbesson sbesson merged commit 2e17a95 into glencoesoftware:master Mar 5, 2026
4 checks passed
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.

Make EventBus Timeout Configurable

2 participants