import java.io.IOException;
import java.io.InputStream;
import java.util.Optional;
+import java.util.concurrent.Callable;
+import java.util.concurrent.ExecutionException;
import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Future;
import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.concurrent.atomic.AtomicReference;
import net.pterodactylus.fcp.AllData;
import net.pterodactylus.fcp.ClientGet;
+import net.pterodactylus.fcp.FcpMessage;
import net.pterodactylus.fcp.FcpUtils.TempInputStream;
import net.pterodactylus.fcp.GetFailed;
import net.pterodactylus.fcp.Priority;
import net.pterodactylus.fcp.ReturnType;
+import com.google.common.util.concurrent.ListenableFuture;
+import com.google.common.util.concurrent.ListeningExecutorService;
+import com.google.common.util.concurrent.MoreExecutors;
+
/**
* Implementation of the {@link ClientGetCommand}.
*
*/
class ClientGetCommandImpl implements ClientGetCommand {
- private final ExecutorService threadPool;
+ private final ListeningExecutorService threadPool;
private final ConnectionSupplier connectionSupplier;
- private String identifier;
private boolean ignoreDataStore;
private boolean dataStoreOnly;
private Long maxSize;
private boolean global;
public ClientGetCommandImpl(ExecutorService threadPool, ConnectionSupplier connectionSupplier) {
- this.threadPool = threadPool;
+ this.threadPool = MoreExecutors.listeningDecorator(threadPool);
this.connectionSupplier = connectionSupplier;
}
@Override
- public ClientGetCommand identifier(String identifier) {
- this.identifier = identifier;
- return this;
- }
-
- @Override
public ClientGetCommand ignoreDataStore() {
ignoreDataStore = true;
return this;
}
@Override
- public Future<Optional<Data>> uri(String uri) {
+ public Executable<Optional<Data>> uri(String uri) {
+ return () -> threadPool.submit(() -> execute(uri));
+ }
+
+ private Optional<Data> execute(String uri) throws InterruptedException, ExecutionException, IOException {
+ ClientGet clientGet = createClientGetCommand(uri);
+ try (ClientGetReplySequence clientGetReplySequence = new ClientGetReplySequence()) {
+ return clientGetReplySequence.send(clientGet).get();
+ }
+ }
+
+ private ClientGet createClientGetCommand(String uri) {
+ String identifier = new RandomIdentifierGenerator().generate();
ClientGet clientGet = new ClientGet(uri, identifier, ReturnType.direct);
if (ignoreDataStore) {
clientGet.setIgnoreDataStore(true);
if (global) {
clientGet.setGlobal(true);
}
- return threadPool.submit(() -> {
- FcpReplySequence<Optional<Data>> replySequence =
- new ClientGetReplySequence();
- return replySequence.send(clientGet).get();
- });
+ return clientGet;
}
private class ClientGetReplySequence extends FcpReplySequence<Optional<Data>> {
private final AtomicBoolean finished = new AtomicBoolean();
private final AtomicBoolean failed = new AtomicBoolean();
- private final String identifier = ClientGetCommandImpl.this.identifier;
-
private String contentType;
private long dataLength;
private InputStream payload;
@Override
protected void consumeAllData(AllData allData) {
- if (allData.getIdentifier().equals(identifier)) {
- synchronized (this) {
- contentType = allData.getContentType();
- dataLength = allData.getDataLength();
- try {
- payload = new TempInputStream(allData.getPayloadInputStream(), dataLength);
- finished.set(true);
- } catch (IOException e) {
- // TODO – logging
- failed.set(true);
- }
+ synchronized (this) {
+ contentType = allData.getContentType();
+ dataLength = allData.getDataLength();
+ try {
+ payload = new TempInputStream(allData.getPayloadInputStream(), dataLength);
+ finished.set(true);
+ } catch (IOException e) {
+ // TODO – logging
+ failed.set(true);
}
}
}
@Override
protected void consumeGetFailed(GetFailed getFailed) {
- if (getFailed.getIdentifier().equals(identifier)) {
- failed.set(true);
- }
- }
-
- @Override
- protected void consumeConnectionClosed(Throwable throwable) {
failed.set(true);
}