Overall this looks good. With regards to bulk mode here, I can't believe we didn't already have that setting. There is another setting too that can drastically help with performance called, EnableContentReponseOnWrite. This gets set on insert/update operations. It essentially bypasses sending the response body back to the client which can drastically reduce network io.
I'm not sure if you want to also include that here in this PR as well. It's implemented in a different place. But if you want, go ahead. I'll approve this anyway.
var options = new ItemRequestOptions
{
EnableContentResponseOnWrite = false
};
Originally posted by @markjbrown in #214 (review)
Overall this looks good. With regards to bulk mode here, I can't believe we didn't already have that setting. There is another setting too that can drastically help with performance called, EnableContentReponseOnWrite. This gets set on insert/update operations. It essentially bypasses sending the response body back to the client which can drastically reduce network io.
I'm not sure if you want to also include that here in this PR as well. It's implemented in a different place. But if you want, go ahead. I'll approve this anyway.
var options = new ItemRequestOptions
{
EnableContentResponseOnWrite = false
};
Originally posted by @markjbrown in #214 (review)