Looks fairly good. I'd remove batching though since that shouldn't be required.
Did you check the logs? Did you get any exception(s)?
Thanks! We figured out that it was not a cache replication issue. We found that in our application code, we were using a non thread safe data structure (which eventually gets stored in Infinispan) and the data structure was getting corrupted occasionally when there are concurrent write operations. We fixed the issue by changing the application code to use a thread safe data structure.
Can you please confirm if there is any functional or performance impact in removing batching? We don't actually use cache store batch operations however we had batching enabled for quite sometime. So just wanted to confirm before removing it.
Batching should not make a diff in your case