dcapwell commented on code in PR #2190:
URL: https://github.com/apache/cassandra/pull/2190#discussion_r1127128930
##########
src/java/org/apache/cassandra/service/accord/AccordService.java:
##########
@@ -226,7 +226,7 @@ public TxnData coordinate(Txn txn, ConsistencyLevel
consistencyLevel)
private static RuntimeException throwTimeout(Txn txn, ConsistencyLevel
consistencyLevel)
{
- throw txn.isWrite() ? new WriteTimeoutException(WriteType.TRANSACTION,
consistencyLevel, 0, 0)
+ throw txn.isWrite() ? new WriteTimeoutException(WriteType.CAS,
consistencyLevel, 0, 0)
Review Comment:
> no support for transaction in the client yet,
correct, java driver takes our `String` and converts to an `Enum`, which
then fails; java driver does not handle unknown types...
> If so, maybe its worth to add a comment.
makes sense, would have avoided this if a comment existed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
