Skip to content

Commit 59587f1

Browse files
committed
Fix based on feedback
1 parent 38311d2 commit 59587f1

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

core/src/main/java/com/scalar/db/storage/cassandra/MutateStatementHandler.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ public ResultSet handle(Operation operation) throws ExecutionException {
4444
return results;
4545
} catch (WriteTimeoutException e) {
4646
if (e.getWriteType() == WriteType.CAS) {
47-
// retry needs to be done if applications need to do the operation exactly
4847
throw new ExecutionException(
4948
CoreError.CASSANDRA_WRITE_TIMEOUT_IN_PAXOS_PHASE_IN_MUTATION.buildMessage(), e);
5049
} else if (e.getWriteType() == WriteType.SIMPLE) {
@@ -54,7 +53,6 @@ public ResultSet handle(Operation operation) throws ExecutionException {
5453
throw new ReadRepairableExecutionException(
5554
CoreError.CASSANDRA_WRITE_TIMEOUT_IN_LEARN_PHASE_IN_MUTATION.buildMessage(), e);
5655
} else {
57-
// retry needs to be done if applications need to do the operation exactly
5856
throw new ExecutionException(
5957
CoreError.CASSANDRA_WRITE_TIMEOUT_SIMPLE_WRITE_OPERATION_FAILED_IN_MUTATION
6058
.buildMessage(),

core/src/test/java/com/scalar/db/storage/cassandra/InsertStatementHandlerTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,8 @@ public void handle_PutWithConditionGivenAndWTEWithSimpleThrown_ShouldThrowExecut
363363
}
364364

365365
@Test
366-
public void handle_PutWithoutConditionGivenAndWTEWithSimpleThrown_ShouldExecutionException() {
366+
public void
367+
handle_PutWithoutConditionGivenAndWTEWithSimpleThrown_ShouldThrowExecutionException() {
367368
// Arrange
368369
put = preparePutWithClusteringKey();
369370
spy = prepareSpiedInsertStatementHandler();

0 commit comments

Comments
 (0)