On Thu Feb 26, 2026 at 4:42 AM JST, Zhi Wang wrote: > On Wed, 25 Feb 2026 22:41:49 +0900 > Eliot Courtney <[email protected]> wrote: > >> Add sync and async command queue API and the type infrastructure to >> know what reply is expected from each `CommandToGsp`. >> > > snip > >> + loop { >> + match self.receive_msg::<M::Reply>(Delta::from_secs(10)) > > It turns out here the timeout is changed to 10s from 5s which was used > in other places. Any problem you encountered during the debugging?
No issues, but it /looked/ to me like the particular values didn't really matter so it would be fine and simpler to have a uniform 10 second timeout for now. Please LMK if you know this is wrong and the 5 second value is important and specifically chosen. I think it's also fine to include the timeout as a parameter and would also fit the convention with receive_msg. Thanks for the review~
