A description of the problem is in the attached BF.

One solution would be to avoid the compiler optimizations by reordering the arguments to the or statements so that the refreshIfNeeded must be executed.

However, it may be a less fragile solution to include the refresh in the createCollectionIfNeeded function so that _lastError is always reset to boost::none, even if a new caller of these functions forgets to call both.

We should consider, especially in the bulk_write_exec where there are many targeters, whether we want all of the statements to execute or whether we want to exit early as part of this ticket.