apache / apache/pulsar-adapters

Thread.sleep used in some unit tests; consider replacing with Awaitility

Open
#23 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
24
Forks
30
PR merge metrics
No merged PRs in 30d

Description

Thread.sleep used in some unit tests (search code for it); consider replacing with Awaitility.

This is a follow up on https://github.com/apache/pulsar-adapters/pull/22#discussion_r654194531

Priority is fairly low IMO - occurences of this are very limited, test run reasonably fast AFAICT, it's unclear what specifically the tests have to wait for etc.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.