Skip to content

[SPARK-52428] Harden client module error handling#24

Open
rafafrdz wants to merge 1 commit intoapache:masterfrom
rafafrdz:hardening-client-error-handling
Open

[SPARK-52428] Harden client module error handling#24
rafafrdz wants to merge 1 commit intoapache:masterfrom
rafafrdz:hardening-client-error-handling

Conversation

@rafafrdz
Copy link
Copy Markdown

Summary

  • Replace unwrap() with ok_or_else() + ? on operation_id and response_id in reattach/release operations
  • Replace unimplemented!() with Err(SparkError::NotYetImplemented(...)) for extension response types and non-Utf8 data types
  • Replace unwrap() on header parsing in middleware with graceful skip + eprintln! warning
  • Improve ChannelBuilder::default() fallback

Test plan

  • cargo build passes
  • cargo test unit tests pass
  • cargo clippy passes
  • cargo fmt -- --check passes

@rafafrdz rafafrdz force-pushed the hardening-client-error-handling branch from 31c516d to 9c01c3f Compare March 29, 2026 08:08
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