[4/9] feat: add process primitives (Pid, ProcessTable, Registry)#108
Closed
unbalancedparentheses wants to merge 8 commits intolambdaclass:mainfrom
Closed
[4/9] feat: add process primitives (Pid, ProcessTable, Registry)#108unbalancedparentheses wants to merge 8 commits intolambdaclass:mainfrom
unbalancedparentheses wants to merge 8 commits intolambdaclass:mainfrom
Conversation
Consolidate the two separate GenServer implementations (async/tokio and threads) into a single implementation with a Backend enum parameter. Breaking change: start() now requires a Backend argument: - Backend::Async - tokio async tasks (default) - Backend::Blocking - tokio's blocking thread pool - Backend::Thread - dedicated OS thread This provides runtime flexibility without code duplication, allowing users to mix different execution backends in the same application. - Add Backend enum to gen_server.rs - Change start() signature to accept Backend parameter - Update all examples and tests - Remove thread-based example crates
Add 10 new tests covering: - Backend enum traits (Default, Copy, Clone, Debug, PartialEq, Eq) - All three backends handle call/cast correctly - Backend::Thread isolates blocking work from async runtime - Multiple backends can run concurrently with independent state - Backend::default() works in start()
Document each backend option with: - Comparison table showing execution model, best use cases, and limitations - Code examples for each backend - Detailed "When to Use" guide with advantages and avoid-when advice - Per-variant documentation with specific use cases
Move files from tasks/ to root and remove threads/ directory: - concurrency/src/tasks/* → concurrency/src/* - Delete concurrency/src/threads/ (replaced by Backend enum) - Update all imports from spawned_concurrency::tasks:: to spawned_concurrency:: The Backend enum (Async, Blocking, Thread) now provides all the functionality previously split between tasks and threads modules, offering a cleaner and more unified API. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add Pid struct with unique process identifiers (AtomicU64) - Add HasPid trait for types that have a process ID - Add ExitReason enum (Normal, Shutdown, Error, Killed) - Add MonitorRef for tracking monitors - Add SystemMessage enum (Down, Exit, Timeout) These are the foundational types for OTP-style process management.
- Add ProcessTable with global process tracking - Implement bidirectional linking (link/unlink) - Implement unidirectional monitoring (monitor/demonitor) - Add trap_exit support for catching linked process exits - Add SystemMessageSender trait for delivering DOWN/EXIT messages - Handle exit propagation to linked processes The process table is the central registry for all running processes and manages the relationships between them.
- Add global Registry for name -> Pid mapping - Implement register/unregister functions - Add whereis for name lookup - Add name_of for reverse lookup (Pid -> name) - Prevent duplicate names and multiple names per process - Add comprehensive tests with mutex for isolation Enables Erlang-style named processes for easier discovery.
5 tasks
unbalancedparentheses
changed the title
Collaborator
|
Closing as this PR depends on #107 which incorrectly deletes threads/. The valuable features (Pid, ProcessTable, Registry) will be reimplemented correctly while preserving both tasks/ and threads/ modules. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Pidstruct with unique process identifiers and Erlang-style display format<0.N>HasPidtrait for types that have a process identifierLinkmodule withMonitorRefandSystemMessagefor process linking/monitoringProcessTablefor global process management, linking, and monitoringRegistryfor name-based process lookup (like Erlang'sregister/2)Features
Hash/Eqfor HashMap keysregister,unregister,whereis,registeredChanges
concurrency/src/pid.rs- Pid struct and HasPid trait (~200 lines)concurrency/src/link.rs- MonitorRef and SystemMessage (~180 lines)concurrency/src/process_table.rs- Global process table (~490 lines)concurrency/src/registry.rs- Process registry (~370 lines)concurrency/src/lib.rs- Export new modulesDepends on: PR #107 (flatten module structure)
Test plan
🤖 Generated with Claude Code