jallum / bedrock / b6edf2721597b2e3285bded8c2b796c010f396d2-PR-49

66%

develop: 65%

Build Type

Pull #49

github

Committed by jallum

Commit Message

Refactor transaction error handling and modernize system architecture

Core Changes:
- Error handling modernization: Replaced exception-based control flow with structured error patterns
  - Converted HCARetryException to {__MODULE__, :retry} throw pattern in HCA allocation
  - Enhanced storage racing with detailed failure categorization and proper error propagation
  - Added StorageError and TransactionError exception types for proper error classification
- Transaction builder enhancements:
  - Renamed Committing module to Finalization for better semantic clarity
  - Added nested transaction support with proper stack management
  - Enhanced failure handling with structured error reasons across all operations
  - Added support for atomic operations and individual key clearing
- Storage layer improvements:
  - Refined error propagation with detailed failure reasons (timeout, unavailable, layout failures)
  - Enhanced storage racing with proper server failure tracking
  - Removed timeout parameter from async streams for better reliability
- Repository layer overhaul:
  - Simplified transaction management with proper retry logic
  - Enhanced error classification between retryable and non-retryable failures
  - Removed complex TransactionManager in favor of direct repo transaction handling
  - Added comprehensive error handling tests
- Directory layer enhancements:
  - Added range() operation to Directory protocol
  - Improved HCA integration with proper transaction context
  - Enhanced prefix allocation with better error handling
  - Fixed root directory handling semantics
- Testing & tooling:
  - Added StorageTelemetry debug module for operation monitoring
  - Comprehensive test coverage for all error handling scenarios
  - Added storage retry tests and error handling validation

Pull Request Pull Request #49: Implement FoundationDB-compatible directory layer with HCA and subspace support

Run Details

276 of 409 new or added lines in 18 files covered. (67.48%)

4 existing lines in 3 files now uncovered.

3687 of 5672 relevant lines covered (65.0%)

653.38 hits per line