Skip to content

Move rkyv RPC support to use CRC32 checksums rather than checkbytes and use stack allocated scratch. #183

Move rkyv RPC support to use CRC32 checksums rather than checkbytes and use stack allocated scratch.

Move rkyv RPC support to use CRC32 checksums rather than checkbytes and use stack allocated scratch. #183

Triggered via pull request September 3, 2023 13:53
Status Failure
Total duration 9m 37s
Artifacts

rust.yml

on: pull_request
Matrix: build
Matrix: check
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 49 warnings
build (ubuntu-latest, stable)
Process completed with exit code 100.
build (windows-latest, stable)
The operation was canceled.
build (macOS-latest, stable)
The operation was canceled.
use of `default` to create a unit struct: datacake-eventual-consistency/src/lib.rs#L586
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/lib.rs:586:38 | 586 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/replication_impl.rs#L274
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/replication_impl.rs:274:49 | 274 | _marker: PhantomData::<MemStore>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/replication_impl.rs#L228
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/replication_impl.rs:228:49 | 228 | _marker: PhantomData::<MemStore>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L218
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:218:42 | 218 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L204
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:204:42 | 204 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L176
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:176:38 | 176 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L149
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:149:38 | 149 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L122
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:122:38 | 122 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L93
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:93:38 | 93 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/replication/poller.rs#L513
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/replication/poller.rs:513:34 | 513 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/replication/poller.rs#L504
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/replication/poller.rs:504:38 | 504 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/replication/poller.rs#L471
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/replication/poller.rs:471:38 | 471 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-node/src/lib.rs#L151
warning: use of `default` to create a unit struct --> datacake-node/src/lib.rs:151:43 | 151 | node_selector: DCAwareSelector::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/keyspace/group.rs#L254
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/keyspace/group.rs:254:69 | 254 | if let Err(e) = state.send(PurgeDeletes(PhantomData::<S>::default())).await { | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default
use of `default` to create a unit struct: datacake-node/src/nodes_selector.rs#L544
warning: use of `default` to create a unit struct --> datacake-node/src/nodes_selector.rs:544:43 | 544 | let mut selector = DCAwareSelector::default(); | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default
this `impl` can be derived: datacake-rpc/src/rkyv_tooling/scratch.rs#L21
warning: this `impl` can be derived --> datacake-rpc/src/rkyv_tooling/scratch.rs:21:1 | 21 | / impl Default for LazyScratch { 22 | | fn default() -> Self { 23 | | Self { 24 | | stack_scratch: StackScratch::default(), ... | 28 | | } 29 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it | 15 + #[derive(Default)] 16 | pub struct LazyScratch { |
use of `default` to create a unit struct: datacake-rpc/src/handler.rs#L119
warning: use of `default` to create a unit struct --> datacake-rpc/src/handler.rs:119:37 | 119 | _msg: PhantomData::<Msg>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default
use of `default` to create a unit struct: datacake-eventual-consistency/src/lib.rs#L586
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/lib.rs:586:38 | 586 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/lib.rs#L529
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/lib.rs:529:38 | 529 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/lib.rs#L469
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/lib.rs:469:38 | 469 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/lib.rs#L404
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/lib.rs:404:38 | 404 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L218
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:218:42 | 218 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L204
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:204:42 | 204 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L176
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:176:38 | 176 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L149
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:149:38 | 149 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L122
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:122:38 | 122 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/rpc/services/consistency_impl.rs#L93
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/rpc/services/consistency_impl.rs:93:38 | 93 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/replication/poller.rs#L513
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/replication/poller.rs:513:34 | 513 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/replication/poller.rs#L504
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/replication/poller.rs:504:38 | 504 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/replication/poller.rs#L471
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/replication/poller.rs:471:38 | 471 | _marker: PhantomData::<S>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs
use of `default` to create a unit struct: datacake-eventual-consistency/src/keyspace/group.rs#L254
warning: use of `default` to create a unit struct --> datacake-eventual-consistency/src/keyspace/group.rs:254:69 | 254 | if let Err(e) = state.send(PurgeDeletes(PhantomData::<S>::default())).await { | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default
use of `default` to create a unit struct: datacake-node/src/lib.rs#L151
warning: use of `default` to create a unit struct --> datacake-node/src/lib.rs:151:43 | 151 | node_selector: DCAwareSelector::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default
this `impl` can be derived: datacake-rpc/src/rkyv_tooling/scratch.rs#L21
warning: this `impl` can be derived --> datacake-rpc/src/rkyv_tooling/scratch.rs:21:1 | 21 | / impl Default for LazyScratch { 22 | | fn default() -> Self { 23 | | Self { 24 | | stack_scratch: StackScratch::default(), ... | 28 | | } 29 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it | 15 + #[derive(Default)] 16 | pub struct LazyScratch { |
use of `default` to create a unit struct: datacake-rpc/src/handler.rs#L119
warning: use of `default` to create a unit struct --> datacake-rpc/src/handler.rs:119:37 | 119 | _msg: PhantomData::<Msg>::default(), | ^^^^^^^^^^^ help: remove this call to `default` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#default_constructed_unit_structs = note: `#[warn(clippy::default_constructed_unit_structs)]` on by default
check (ubuntu-latest, stable)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, Swatinem/rust-cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
check (ubuntu-latest, stable)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
check (ubuntu-latest, stable)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
check (ubuntu-latest, stable)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (ubuntu-latest, stable)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
check (macOS-latest, stable)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, Swatinem/rust-cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
check (macOS-latest, stable)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
check (macOS-latest, stable)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
check (macOS-latest, stable)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build (windows-latest, stable)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (macOS-latest, stable)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
check (windows-latest, stable)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, Swatinem/rust-cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
check (windows-latest, stable)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
check (windows-latest, stable)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
check (windows-latest, stable)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/