Yes, if it is a constant.


On 30 August 2013 12:05, Pedro Ruivo <notifications@github.com> wrote:

> In
> cachestore/jdbc/src/main/java/org/infinispan/loaders/jdbc/binary/JdbcBinaryCacheStore.java:
>
> > @@ -378,7 +378,7 @@ public void purgeInternal() throws CacheLoaderException {
> >                    ps.setString(3, bucket.getBucketIdAsString());
> >                    ps.addBatch();
> >                    updateCount++;
> > -                  if (updateCount % batchSize == 0) {
> > +                  if (updateCount % PURGE_BATCH_SIZE == 0) {
>
> is the % correct? or the compiler is smart enough to optimize it?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/2035/files#r6083695>
> .
>