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-unsubscr...@beam.apache.org

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


Reply via email to