Skip to content

[proto] add RejectedTxIds to NotificationResponse#89

Merged
liran-funaro merged 1 commit intohyperledger:mainfrom
cendhu:sidecar-notify-reject
Mar 9, 2026
Merged

[proto] add RejectedTxIds to NotificationResponse#89
liran-funaro merged 1 commit intohyperledger:mainfrom
cendhu:sidecar-notify-reject

Conversation

@cendhu
Copy link
Contributor

@cendhu cendhu commented Mar 6, 2026

Type of change

  • New feature

Description

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.

Related issues

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>
@cendhu cendhu requested a review from liran-funaro March 9, 2026 09:50
Copy link
Contributor

@liran-funaro liran-funaro left a comment

Choose a reason for hiding this comment

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

LGTM.

@liran-funaro liran-funaro merged commit 897c999 into hyperledger:main Mar 9, 2026
7 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.

2 participants