fix(nats): use fetch().expires(30s) instead of messages() — without expires NATS returns empty immediately
Some checks failed
lint / lint (push) Has been cancelled
test / unit (push) Has been cancelled
test / integration (push) Has been cancelled
test / integration (pull_request) Has been cancelled
test / unit (pull_request) Has been cancelled
lint / lint (pull_request) Has been cancelled
Some checks failed
lint / lint (push) Has been cancelled
test / unit (push) Has been cancelled
test / integration (push) Has been cancelled
test / integration (pull_request) Has been cancelled
test / unit (pull_request) Has been cancelled
lint / lint (pull_request) Has been cancelled
This commit is contained in:
@@ -137,10 +137,19 @@ impl MessageSource for NatsMessageSource {
|
||||
tracing::info!("NATS pull consumer ready");
|
||||
|
||||
loop {
|
||||
let mut messages = match consumer.messages().await {
|
||||
// fetch().expires() keeps the request open server-side until messages arrive
|
||||
// (up to 30 s), then the batch ends and we loop. Without expires, the fetch
|
||||
// returns immediately with zero messages when the queue is empty.
|
||||
let mut messages = match consumer
|
||||
.fetch()
|
||||
.max_messages(100)
|
||||
.expires(std::time::Duration::from_secs(30))
|
||||
.messages()
|
||||
.await
|
||||
{
|
||||
Ok(m) => m,
|
||||
Err(e) => {
|
||||
tracing::error!("NATS consumer.messages() failed: {e}");
|
||||
tracing::error!("NATS fetch failed: {e}");
|
||||
let _ = tx.send(Err(DomainError::Internal(e.to_string()))).await;
|
||||
return;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user