film42 / sidekiq-rs

A port of sidekiq to rust using tokio
MIT License
95 stars 10 forks source link

Quick optimization pass #2

Closed film42 closed 2 years ago

film42 commented 2 years ago

This removes unnecessary clone calls and makes Job a reference through the middleware interface. It might make sense to have args: JsonValue be a reference as well. The reason I'm not sure yet is that clone is required to call serde_json::from_value but not all jobs take arguments.