Skip to content

conformance: remove try-except blocks from server impl#85

Merged
tsubakiky merged 1 commit intomainfrom
fix-conformance-server-try
May 14, 2025
Merged

conformance: remove try-except blocks from server impl#85
tsubakiky merged 1 commit intomainfrom
fix-conformance-server-try

Conversation

@tsubakiky
Copy link
Copy Markdown
Collaborator

@tsubakiky tsubakiky commented May 14, 2025

This pull request primarily focuses on simplifying error handling in the conformance/server.py file by removing redundant try-except blocks, alongside a few other minor updates to improve maintainability and clean up unused configurations.

Error handling simplification in conformance/server.py:

  • Removed redundant try-except blocks for ConnectError and generic exceptions in multiple asynchronous methods, including Unary, IdempotentUnary, ClientStream, ServerStream, and BidiStream. This change assumes that unhandled exceptions will propagate naturally, simplifying the code and reducing unnecessary boilerplate. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Configuration and logging cleanup:

  • Removed the duplicate metadata test case entry from conformance/run-testcase.txt to avoid redundancy.
  • Removed the unused "Deadline Propagation" configuration from conformance/server_known_failing.yaml.
  • Refactored logging setup in conformance/server_runner.py to streamline logger initialization and ensure consistent logging behavior.

@tsubakiky tsubakiky changed the title conformance: small fix server impl conformance: remove try-except blocks server impl May 14, 2025
@tsubakiky tsubakiky changed the title conformance: remove try-except blocks server impl conformance: remove try-except blocks from server impl May 14, 2025
@tsubakiky tsubakiky merged commit 17205d3 into main May 14, 2025
2 checks passed
@tsubakiky tsubakiky deleted the fix-conformance-server-try branch May 14, 2025 11:41
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.

1 participant