Skip to content

[Python] Log error details when ExecuteBatchError occurs (at DEBUG level)#40921

Closed
chandra-siri wants to merge 1 commit intogrpc:masterfrom
chandra-siri:log_exception_details
Closed

[Python] Log error details when ExecuteBatchError occurs (at DEBUG level)#40921
chandra-siri wants to merge 1 commit intogrpc:masterfrom
chandra-siri:log_exception_details

Conversation

@chandra-siri
Copy link
Contributor

@chandra-siri chandra-siri commented Oct 16, 2025

Log error details when ExecuteBatchError occurs. The error is logged with DEBUG severity level. Message example:

Failed to receive any message from Core: Failed grpc_call_start_batch: 8 with grpc_call_error value: 'GRPC_CALL_ERROR_TOO_MANY_OPERATIONS

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 16, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: chandra-siri / name: Chandra Shekhar Sirimala (316de3d)

@chandra-siri chandra-siri changed the title grpc-python: Log error details when ExecuteBatchError occurs [Python] [Chore]: Log error details when ExecuteBatchError occurs Oct 16, 2025
@chandra-siri chandra-siri changed the title [Python] [Chore]: Log error details when ExecuteBatchError occurs [Python] [Chore] Log error details when ExecuteBatchError occurs Oct 16, 2025
@asheshvidyut asheshvidyut added the release notes: no Indicates if PR should not be in release notes label Oct 16, 2025
@chandra-siri chandra-siri marked this pull request as ready for review October 16, 2025 08:29
Copy link
Member

@sergiitk sergiitk left a comment

Choose a reason for hiding this comment

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

I'm OK with this change overall.

@chandra-siri: Were you able to test this code path, manually of otherwise? Could you please share a sample of the final log message produce?

@sergiitk sergiitk removed the request for review from asheshvidyut October 16, 2025 21:55
@sergiitk sergiitk added release notes: yes Indicates if PR needs to be in release notes kokoro:run and removed release notes: no Indicates if PR should not be in release notes labels Oct 16, 2025
@sreenithi
Copy link
Contributor

LGTM as well. I've re-triggered the Python and sanity tests as they seemed to be flaky failures

@chandra-siri
Copy link
Contributor Author

@chandra-siri: Were you able to test this code path, manually of otherwise? Could you please share a sample of the final log message produce?

yes, I was able to test the code path. Here's the output sample.

2025-10-21 17:28:58,069 - grpc._cython.cygrpc:378 - DEBUG - Failed to receive any message from Core: Failed grpc_call_start_batch: 8 with grpc_call_error value: 'GRPC_CALL_ERROR_TOO_MANY_OPERATIONS

also attached the image.
see
Screenshot 2025-10-21 at 11 07 49 PM

@chandra-siri chandra-siri requested a review from sergiitk October 21, 2025 17:39
@sergiitk sergiitk changed the title [Python] [Chore] Log error details when ExecuteBatchError occurs [Python] Log error details when ExecuteBatchError occurs (at DEBUG level) Oct 23, 2025
@chandra-siri
Copy link
Contributor Author

Not sure why copybara-service bot closed it ?

any idea @sergiitk , @sreenithi ?

@sreenithi
Copy link
Contributor

oh nothing to worry, that's the way our process works. Your changes were submitted in this commit by copybara and the PR is closed.

Your changes are now live in master: https://github.com/grpc/grpc/blob/master/src/python/grpcio/grpc/_cython/_cygrpc/aio/callback_common.pyx.pxi#L133

sreenithi pushed a commit to sreenithi/grpc that referenced this pull request Nov 26, 2025
…vel) (grpc#40921)

Log error details when `ExecuteBatchError` occurs. The error is logged with DEBUG severity level. Message example:

> `Failed to receive any message from Core: Failed grpc_call_start_batch: 8 with grpc_call_error value: 'GRPC_CALL_ERROR_TOO_MANY_OPERATIONS`

Closes grpc#40921

COPYBARA_INTEGRATE_REVIEW=grpc#40921 from chandra-siri:log_exception_details 316de3d
PiperOrigin-RevId: 823347363
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/Python release notes: yes Indicates if PR needs to be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants