You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2021/09/01 05:03:02 UTC

[GitHub] [beam] pareshsarafmdb commented on pull request #14927: [BEAM-12400] MongoDBIO support for update within documents

pareshsarafmdb commented on pull request #14927:
URL: https://github.com/apache/beam/pull/14927#issuecomment-909888653


   > > Just added minor stylistic comments.
   > > I still wonder: What happens when users want to update the whole old record into the whole new record?
   > > Finally, I have one more question about a later improvement to consider for this IO - is it worth thinking later about passing a lambda to choose the destination collection?
   > 
   > @pabloem If I understand what if user wants to update with entire source document into a destination document - In that case you have a overloaded method which just takes update operator and dest field. This does full document update. Did I make sense?
   
   Yes thats right,


-- 
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: github-unsubscribe@beam.apache.org

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