pjfanning commented on PR #63: URL: https://github.com/apache/incubator-pekko-persistence-dynamodb/pull/63#issuecomment-1639853060
> > @mdedetrich only pekko-http has such complicated code for dealing with the pekko snapshots. Most other repos, we prefer to just have a pekkoVersion which means we control exactly which snapshot (or release) to use. > > Actually another repo has it as well, i.e. see https://github.com/apache/incubator-pekko-persistence-dynamodb/blob/main/project/PekkoDependency.scala . > > I am not against removing it but I would like to get feedback from either @jrudolph or @raboof on the ramifications of it. I get the impression that they (as in Lightbend/Akka) were actually planning to slowly integrated this code for dealing with snapshots into all of the modules but because of engineering capacity they only managed akka-http/akka-persistence-dynamodb @mdedetrich you are pointing back at this repo - this PR is in incubator-pekko-persistence-dynamodb and the supposed extra copy of this file is in incubator-pekko-persistence-dynamodb - there is only 1 file -- 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]
