Skip to content

Commit 7768606

Browse files
committed
Clarify consumer log message when chunk reservation expired
The original message was a "todo"/future idea from when opsqueue was more actively being developed. It has a high severity and it sounds actionable to users reading the logs, whereas there isn't really any way they can change the handling. This is a little misleading, so this commit changes the message to be clearer about what's going on.
1 parent 47904d2 commit 7768606

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

opsqueue/src/consumer/client.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -282,10 +282,9 @@ impl Client {
282282

283283
},
284284
ServerToClientMessage::Async(msg) => {
285-
// Handle a message from the server that was not associated with an earlier request
286285
match msg {
287-
AsyncServerToClientMessage::ChunkReservationExpired(_chunk_id) => {
288-
tracing::error!("Client could cancel execution of current work, but this is not implemented yet.");
286+
AsyncServerToClientMessage::ChunkReservationExpired(chunk_id) => {
287+
tracing::debug!("Server indicated that we took too long with {chunk_id:?}, and now our reservation has expired.");
289288
},
290289
}
291290
}

0 commit comments

Comments
 (0)