I did not detect any apparent double boxing issues here however I refactored the code to use the custom AsyncValueFuture enum and eliminate the unnecessary async move closure. This is my first contribution here - all cargo tests passed however the test.book failed with some strange error about the dataloader crate which seems unrelated.
P.S. I am new to rust and snooping around for real-world practice so please advise me if I've made mistakes or missed something! Thank you in advance and hope to contribute more in the future!
Closes #820
I did not detect any apparent double boxing issues here however I refactored the code to use the custom
AsyncValueFutureenum and eliminate the unnecessaryasync moveclosure. This is my first contribution here - allcargo tests passed however thetest.bookfailed with some strange error about thedataloadercrate which seems unrelated.P.S. I am new to rust and snooping around for real-world practice so please advise me if I've made mistakes or missed something! Thank you in advance and hope to contribute more in the future!