Re: [PR] try not to use stored failed future results and therefore allow recovery [incubator-pekko-persistence-cassandra]

["pjfanning (via GitHub)" <gi...@apache.org>]

pjfanning commented on PR #106:
URL: https://github.com/apache/incubator-pekko-persistence-cassandra/pull/106#issuecomment-1775659066

   @mdedetrich what do you think of this? Do you think it is safe as is?
   
   My feeling is that the LazyFutureEval hides the retry semantics from the API user - so we can tweak the retry behaviour in a future release if this PR's approach does not retry actively enough - but the change would not need to affect the API and binary compatibility.
   
   We might be in good position to release a 1.0.1 patch soon - maybe after we get Pekko Connectors 1.0.1 released.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org