You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "spangaer (via GitHub)" <gi...@apache.org> on 2023/04/21 09:53:44 UTC

[GitHub] [incubator-pekko-persistence-dynamodb] spangaer commented on pull request #12: Add setSelect(ALL_ATTRIBUTES) to retrieve snapshots

spangaer commented on PR #12:
URL: https://github.com/apache/incubator-pekko-persistence-dynamodb/pull/12#issuecomment-1517577048

   We'll address point 1 upon rebase.
   
   Point 2, while I don't mind the version bump, I don't think there will be a performance impact.
   
   It didn't work without projecting all before. If you project all, it will keep on working in the same way it did before (inside Dynamo). Only if you don't project all with will behave differently on on the Dynamo side, but that way of working didn't work before this fix.


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