Skip to content

Commit 0b00d3b

Browse files
committed
[proto] add RejectedTxIds to NotificationResponse
Add a new RejectedTxIds message and field to NotificationResponse to allow the notifier to report rejected transaction IDs with a reason (e.g., resource exhausted, request too large). This enables graceful rejection without terminating the stream. Signed-off-by: Senthilnathan <cendhu@gmail.com>
1 parent 7df0f34 commit 0b00d3b

File tree

2 files changed

+91
-17
lines changed

2 files changed

+91
-17
lines changed

api/committerpb/notify.pb.go

Lines changed: 84 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

api/committerpb/notify.proto

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,4 +37,11 @@ message NotificationRequest {
3737
message NotificationResponse {
3838
repeated TxStatus tx_status_events = 1; // List of transaction status events.
3939
repeated string timeout_tx_ids = 2; // List of timeout events.
40+
RejectedTxIds rejected_tx_ids = 3; // List of rejected transaction IDs with a reason.
41+
}
42+
43+
// RejectedTxIds contains a list of transaction IDs that were rejected along with the reason.
44+
message RejectedTxIds {
45+
repeated string tx_ids = 1; // List of rejected transaction IDs.
46+
string reason = 2; // The reason for rejection.
4047
}

0 commit comments

Comments
 (0)