Commits on Aug 15, 2023

1. make Acker require Send+Sync, re-export redis connection manager …

   While updating bridge to use omniqueue, the usages already in place
   work, mostly as-is, with these two changes.
   
   WRT the redis re-export, rust was not able to infer to generic type for
   a Producer or Consumer and required that I specify the type for `R`.
   Rather than adding a dep on bb8 in bridge, it would be nicer to just
   have it re-exported.
   
   WRT `Acker`, bridge expected to run a consumer loop then hand each
   `Delivery` off to an async "handler" function. This failed to compile
   as-is citing the lack of `Send+Sync`. Making `Acker` a supertrait of
   `Send+Sync` was trivial since all the existing implementations were
   naturally `Send+Sync`, so this is probably not controversial.

   

   svix-onelson committed Aug 15, 2023
   Configuration menu

   • View commit details
   • Copy full SHA for ffd0d6a
   • Browse repository at this point

   Copy the full SHA

   ffd0d6a View commit details

   Browse the repository at this point in the history