/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.apache.jackrabbit.oak.plugins.document;

import java.util.List;

import com.google.common.collect.Lists;

import org.apache.jackrabbit.oak.plugins.document.util.Utils;
import org.junit.Test;

import static org.apache.jackrabbit.oak.plugins.document.Collection.NODES;
import static org.apache.jackrabbit.oak.plugins.document.util.Utils.getIdFromPath;
import static org.apache.jackrabbit.oak.plugins.document.util.Utils.getKeyLowerLimit;
import static org.apache.jackrabbit.oak.plugins.document.util.Utils.getKeyUpperLimit;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assume.assumeFalse;

public class ConcurrentQueryAndInvalidateIT extends AbstractMultiDocumentStoreTest {

    public ConcurrentQueryAndInvalidateIT(DocumentStoreFixture dsf) {
        super(dsf);
        assumeFalse(dsf instanceof DocumentStoreFixture.RDBFixture);
    }

    protected static final int NUM_NODES = 50;

    private volatile long counter;

    @Test
    public void cacheUpdate() throws Exception {
        cacheUpdate(false);
    }

    @Test
    public void cacheUpdateInvalidateAll() throws Exception {
        cacheUpdate(true);
    }

    private void cacheUpdate(final boolean invalidateAll) throws Exception {
        Revision r = newRevision();
        List<UpdateOp> ops = Lists.newArrayList();
        List<String> ids = Lists.newArrayList();
        for (int i = 0; i < NUM_NODES; i++) {
            String id = Utils.getIdFromPath("/node-" + i);
            ids.add(id);
            UpdateOp op = new UpdateOp(id, true);
            op.set(Document.ID, id);
            NodeDocument.setLastRev(op, r);
            ops.add(op);
            removeMe.add(id);
        }
        ds1.remove(NODES, ids);
        ds1.create(NODES, ops);

        for (int i = 0; i < 200; i++) {
            Thread q = new Thread(new Runnable() {
                @Override
                public void run() {
                    // query docs on ds1
                    queryDocuments();
                }
            });
            Thread u = new Thread(new Runnable() {
                @Override
                public void run() {
                    // update docs on ds2
                    Iterable<String> ids = updateDocuments();
                    // invalidate docs on ds1
                    if (invalidateAll) {
                        ds1.invalidateCache();
                    } else {
                        ds1.invalidateCache(ids);
                    }
                }
            });
            q.start();
            u.start();
            q.join();
            u.join();
            for (int j = 0; j < NUM_NODES; j++) {
                NodeDocument doc = ds1.find(NODES, Utils.getIdFromPath("/node-" + j));
                assertNotNull(doc);
                assertEquals("Unexpected revision timestamp for " + doc.getId(),
                        counter, doc.getLastRev().get(1).getTimestamp());
            }
        }
    }

    private Revision newRevision() {
        return new Revision(++counter, 0, 1);
    }

    private void queryDocuments() {
        ds1.query(NODES, getKeyLowerLimit("/"), getKeyUpperLimit("/"), 100);
    }

    private Iterable<String> updateDocuments() {
        List<String> ids = Lists.newArrayList();
        for (int i = 0; i < NUM_NODES; i++) {
            ids.add(getIdFromPath("/node-" + i));
        }
        UpdateOp op = new UpdateOp("foo", false);
        NodeDocument.setLastRev(op, newRevision());
        ds2.update(NODES, ids, op);
        return ids;
    }
}

@@ -1838,16 +1838,33 @@
 heUpdate
+InvalidateWithIDs
 () throw
@@ -1901,185 +1901,898 @@
 ate(
-false);%0A    %7D%0A%0A    @Test%0A    public void cacheUpdateInvalidateAll() throws Exception %7B%0A        cacheUpdate(true);%0A    %7D%0A%0A    private void cacheUpdate(final boolean invalidateAll
+new Invalidate() %7B%0A            @Override%0A            public void perform(DocumentStore store, Iterable%3CString%3E ids) %7B%0A                store.invalidateCache(ids);%0A            %7D%0A        %7D);%0A    %7D%0A%0A    @Test%0A    public void cacheUpdateInvalidateAll() throws Exception %7B%0A        cacheUpdate(new Invalidate() %7B%0A            @Override%0A            public void perform(DocumentStore store, Iterable%3CString%3E ids) %7B%0A                store.invalidateCache();%0A            %7D%0A        %7D);%0A    %7D%0A%0A    @Test%0A    public void cacheUpdateInvalidateIndividual() throws Exception %7B%0A        cacheUpdate(new Invalidate() %7B%0A            @Override%0A            public void perform(DocumentStore store, Iterable%3CString%3E ids) %7B%0A                for (String id : ids) %7B%0A                    store.invalidateCache(NODES, id);%0A                %7D%0A            %7D%0A        %7D);%0A    %7D%0A%0A    private void cacheUpdate(final Invalidate inv
 ) th
@@ -3889,175 +3889,29 @@
    i
-f (invalidateAll) %7B%0A                        ds1.invalidateCache();%0A                    %7D else %7B%0A                        ds1.invalidateCache(ids);%0A                    %7D
+nv.perform(ds1, ids);
 %0A   
@@ -4967,11 +4967,118 @@
 s;%0A    %7D
+%0A%0A    private interface Invalidate %7B%0A        void perform(DocumentStore store, Iterable%3CString%3E ids);%0A    %7D
 %0A%7D%0A

Improve test coverage on DocumentStore for concurrent query and invalidate