databendlabs / openraft / 11339553314

90%

main: 88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

Improve: remove `Copy` bound from `NodeId`

The `NodeId` type is currently defined as:

```rust
type NodeId: .. + Copy + .. + 'static;
```

This commit removes the `Copy` bound from `NodeId`.
This modification will allow the use of non-`Copy` types as `NodeId`,
providing greater flexibility for applications that prefer
variable-length strings or other non-`Copy` types for node
identification.

This change maintain compatibility by updating derived `Copy`
implementations with manual implementations:

```rust
// Before
#[derive(Copy...)]
pub struct LogId<NID: NodeId> {}

// After
impl<NID: Copy> Copy for LogId<NID> {}
```

Run Details

117 of 122 new or added lines in 27 files covered. (95.9%)

22 existing lines in 5 files now uncovered.

8132 of 9733 relevant lines covered (83.55%)

22.05 hits per line