Skip to content

Commit

Permalink
Merge pull request #285 from alex268/master
Browse files Browse the repository at this point in the history
Removed unncecessary checks
  • Loading branch information
alex268 authored Jun 19, 2024
2 parents 6374c43 + d9bbb2e commit 2e76797
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions core/src/main/java/tech/ydb/core/operation/OperationImpl.java
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,6 @@ public String toString() {

@Override
public CompletableFuture<Status> cancel() {
if (value != null) {
return CompletableFuture.completedFuture(ReadyOperation.ALREADY_DONE_STATUS);
}

GrpcRequestSettings settings = GrpcRequestSettings.newBuilder().build();
OperationProtos.CancelOperationRequest request = OperationProtos.CancelOperationRequest.newBuilder()
.setId(id)
Expand All @@ -80,10 +76,6 @@ public CompletableFuture<Status> cancel() {

@Override
public CompletableFuture<Status> forget() {
if (value != null) {
return CompletableFuture.completedFuture(ReadyOperation.ALREADY_DONE_STATUS);
}

GrpcRequestSettings settings = GrpcRequestSettings.newBuilder().build();
OperationProtos.ForgetOperationRequest request = OperationProtos.ForgetOperationRequest.newBuilder()
.setId(id)
Expand Down

0 comments on commit 2e76797

Please sign in to comment.