/*
 * Licensed to Crate under one or more contributor license agreements.
 * See the NOTICE file distributed with this work for additional
 * information regarding copyright ownership.  Crate 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.
 *
 * However, if you have executed another commercial license agreement
 * with Crate these terms will supersede the license and you may use the
 * software solely pursuant to the terms of the relevant commercial
 * agreement.
 */

package io.crate.data;

import io.crate.concurrent.CompletableFutures;

import java.util.concurrent.CompletionStage;

public class CloseAssertingBatchIterator implements BatchIterator {

    private final BatchIterator delegate;
    private volatile boolean closed = false;

    public CloseAssertingBatchIterator(BatchIterator delegate) {
        this.delegate = delegate;
    }

    @Override
    public void moveToStart() {
        raiseIfClosed();
        delegate.moveToStart();
    }

    @Override
    public boolean moveNext() {
        raiseIfClosed();
        return delegate.moveNext();
    }

    @Override
    public void close() {
        closed = true;
        delegate.close();
    }

    @Override
    public CompletionStage<?> loadNextBatch() {
        if (closed) {
            return CompletableFutures.failedFuture(new IllegalStateException("Iterator is closed"));
        }
        return delegate.loadNextBatch();
    }

    @Override
    public boolean allLoaded() {
        raiseIfClosed();
        return delegate.allLoaded();
    }

    @Override
    public Row currentRow() {
        raiseIfClosed();
        return delegate.currentRow();
    }

    private void raiseIfClosed() {
        if (closed) {
            throw new IllegalStateException("Iterator is closed");
        }
    }
}

/*
 * Licensed to Crate under one or more contributor license agreements.
 * See the NOTICE file distributed with this work for additional
 * information regarding copyright ownership.  Crate 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.
 *
 * However, if you have executed another commercial license agreement
 * with Crate these terms will supersede the license and you may use the
 * software solely pursuant to the terms of the relevant commercial
 * agreement.
 */

package io.crate.data;

import io.crate.concurrent.CompletableFutures;

import java.util.concurrent.CompletionStage;

public class CloseAssertingBatchIterator implements BatchIterator {

    private final BatchIterator delegate;
    private boolean closed = false;

    public CloseAssertingBatchIterator(BatchIterator delegate) {
        this.delegate = delegate;
    }

    @Override
    public void moveToStart() {
        raiseIfClosed();
        delegate.moveToStart();
    }

    @Override
    public boolean moveNext() {
        raiseIfClosed();
        return delegate.moveNext();
    }

    @Override
    public void close() {
        closed = true;
        delegate.close();
    }

    @Override
    public CompletionStage<?> loadNextBatch() {
        if (closed) {
            return CompletableFutures.failedFuture(new IllegalStateException("Iterator is closed"));
        }
        return delegate.loadNextBatch();
    }

    @Override
    public boolean allLoaded() {
        raiseIfClosed();
        return delegate.allLoaded();
    }

    @Override
    public Row currentRow() {
        raiseIfClosed();
        return delegate.currentRow();
    }

    private void raiseIfClosed() {
        if (closed) {
            throw new IllegalStateException("Iterator is closed");
        }
    }
}

Change BatchIteratorCollector.kill to not abuse close for killing