Skip to content

[SPARK-52428] Harden miscellaneous module error handling#27

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

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

Conversation

@rafafrdz
Copy link
Copy Markdown

Summary

  • types.rs: Change from_str_name() to return Result<DataType, SparkError>, replace interval unimplemented!() with safe fallbacks
  • storage.rs: Replace unimplemented!() with StorageLevel::None default
  • catalog.rs: Replace unimplemented!() with Err(SparkError::NotYetImplemented(...))
  • conf.rs: Replace .first().unwrap() with .ok_or_else()
  • plan.rs: Improve panic message, eliminate unwrap via pattern matching
  • functions/mod.rs: Replace unimplemented!() with todo!() in dead-code functions

Test plan

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

@rafafrdz rafafrdz force-pushed the hardening-misc-error-handling branch from 8a891fd to 4a9351d 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