[Python Misc] Revert change to print backtrace in server (v1.60.x backport) (#35462)

XuanWang-Amos · web-flow · commit 06037ca850cd · 2024-01-10T11:07:00.000-08:00
Backport of #34877 to v1.60.x. --- Fix: #34853 In order to make debugging easier, we have begun printing backtraces in servers. However, this change has the unintended consequence of printing errors to stderr by default, which may not be expected by some users. This PR reverts the change. We recommend that users set up a logging sink if they want to see errors. We will add this to our documentation later.
diff --git a/src/python/grpcio/grpc/_server.py b/src/python/grpcio/grpc/_server.py
@@ -579,7 +579,7 @@ def _call_behavior(
                                 exception.__traceback__,
                             )
                         )
-                    traceback.print_exc()
+                        traceback.print_exc()
                     _LOGGER.exception(details)
                     _abort(
                         state,

Original file line number	Diff line number	Diff line change
`@@ -579,7 +579,7 @@ def _call_behavior(`
`579`	`579`	`exception.__traceback__,`
`580`	`580`	`)`
`581`	`581`	`)`
`582`		`- traceback.print_exc()`
	`582`	`+ traceback.print_exc()`
`583`	`583`	`_LOGGER.exception(details)`
`584`	`584`	`_abort(`
`585`	`585`	`state,`