davsclaus commented on PR #14241:
URL: https://github.com/apache/camel/pull/14241#issuecomment-2164397267

   This will not work with concurrent batch
   
   from a to azure
   from b to azure
   
   the array list should likely be a concurrent list, and when you flush then 
you would need some kind of synchronization to avoid a race condition where the 
list is clear and a message is added at the same time, could lead to lost 
message.
   
   Also this batching is okay for mixing concurrent messages in the same batch 
as they are also individual records being send together to azure - to speedup 
this.
   
   There is no upper size on azure, what if the batch has 50.000 elemenents ?


-- 
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: commits-unsubscr...@camel.apache.org

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

Reply via email to