Skip to content

Conversation

@reiase
Copy link
Contributor

@reiase reiase commented Jan 12, 2026

Overview:

Details:

Where should the reviewer start?

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

…ganizing configuration and handling logic

- Deleted the `system.rs` file, consolidating its contents into new modules for better organization.
- Introduced `config.rs` for actor system configuration, including `SystemConfig` and `SpawnOptions`.
- Added `handle.rs` for actor runtime statistics and local actor handling.
- Created `handler.rs` for HTTP/2 message handling, improving the structure of message dispatching.
- Updated the `lib.rs` file to reflect the removal of the `system.rs` module and adjusted public exports accordingly.
- Enhanced overall code clarity and maintainability by restructuring the actor system's components.
- Moved member-related tests from `cluster_tests.rs` to a new dedicated file `member_tests.rs` for better organization and clarity.
- Updated `mod.rs` to include the new `member_tests` module.
- Enhanced documentation in test files to clarify the purpose and structure of tests related to cluster functionality.
- Removed redundant tests from `cluster_tests.rs` to streamline the testing process and focus on integration tests.
- Added detailed documentation on actor-level restarts, including parameters for `@remote` decorator such as `restart_policy`, `max_restarts`, and backoff settings.
- Included example code demonstrating the usage of actor-level restarts.
- Updated security documentation to highlight risks associated with Python Pickle serialization and provided production guidance for enabling mTLS and network isolation.
- Translated new content into Chinese to maintain consistency across language versions.
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 67.91237% with 249 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
crates/pulsing-actor/src/system/mod.rs 77.27% 110 Missing ⚠️
crates/pulsing-actor/src/system/handler.rs 48.62% 56 Missing ⚠️
crates/pulsing-actor/src/system/config.rs 40.25% 46 Missing ⚠️
crates/pulsing-actor/src/system/runtime.rs 70.21% 28 Missing ⚠️
crates/pulsing-actor/src/system/handle.rs 25.00% 9 Missing ⚠️
Files with missing lines Coverage Δ
crates/pulsing-actor/src/cluster/member.rs 100.00% <ø> (ø)
crates/pulsing-actor/src/system_actor/messages.rs 0.00% <ø> (-92.31%) ⬇️
crates/pulsing-actor/src/system_actor/mod.rs 72.62% <ø> (+1.54%) ⬆️
crates/pulsing-actor/src/system/handle.rs 25.00% <25.00%> (ø)
crates/pulsing-actor/src/system/runtime.rs 70.21% <70.21%> (ø)
crates/pulsing-actor/src/system/config.rs 40.25% <40.25%> (ø)
crates/pulsing-actor/src/system/handler.rs 48.62% <48.62%> (ø)
crates/pulsing-actor/src/system/mod.rs 77.27% <77.27%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@reiase reiase merged commit dc8234b into main Jan 12, 2026
12 of 13 checks passed
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.

3 participants