[GitHub] [incubator-pekko-persistence-dynamodb] pjfanning commented on pull request #19: test with newer akka

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

pjfanning commented on PR #19:
URL: https://github.com/apache/incubator-pekko-persistence-dynamodb/pull/19#issuecomment-1426895193

   @spangaer would you be able to review this at some stage?
   
   * I started https://github.com/apache/incubator-pekko-persistence-dynamodb/pull/17 and discovered some tests were broken
   * this repo currently uses akka 2.5 in its build/tests - pekko is based on akka 2.6
   * it appears from this experimental PR that the upgrade to akka 2.6 is the main cause of the test failures
   * akka 2.6 testkit relies on a newer version of scalatest/scalactic and I had to modify 1 test to get it to compile and pass
   * I had to enable java serialization in akka for 3 tests (akka 2.6 must have disabled this feature for security reasons)
   * there is CurrentPersistenceIdsSpec which is still broken - it stackoverflows
   
   If you could have a look that would be great. Any pointers on how to fix CurrentPersistenceIdsSpec would be appreciated.


-- 
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