mdedetrich commented on PR #63: URL: https://github.com/apache/incubator-pekko-persistence-dynamodb/pull/63#issuecomment-1639882670
Well firstly, I don't know why its so urgent to merge this PR today. pekko-http is being released currently, not pekko-persistence-dynamodb. Secondly we have had enough regressions/problems trying to change/remove existing stuff which has actually ended up wasting more of our time, not less (and personally this is starting to get on my nerves). At least for the first release we should default to being conservative and diligent. I will try and have a look later today, the reason why that code is most likely not working is because of a trivial/single line change that was done because there wasn't a pekko full release up until now -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
