Reuben Bond

Results 468 comments of Reuben Bond

Great, thanks again. We'll likely merge this, then. @Costo would you be happy to open a PR based on your change?

@rwkarg that is very odd. Are you saying that `INSERT ... IF NOT EXISTS` is not supported under an LWT? They use this in their docs example: https://docs.datastax.com/en/cql-oss/3.3/cql/cql_using/useInsertLWT.html. I may...

Given we have a version check, can the Cassandra implementation fail the insert if it knows that the version row is there already? The version check should be all we...

@rkargMsft The one remaining thing: I think we should re-enable the commented-out test and implement [this logic](https://github.com/dotnet/orleans/pull/8925#issuecomment-2049969334) within `CassandraClusteringTable`, even if it's duplicating the pattern which `MembershipTableManager` already implements (and...

I pushed an update to re-enable that test & add per-row ETags for silo entries

@yoDon I believe this is by design and there's nothing to fix or improve here except as noted above, we may include codecs for System.Text.Json & Newtonsoft internal types in...

@ilya-girman do you have Microsoft.Orleans.Sdk installed in your test project?

@jbusuttil83 once #9127 is merged, that scenario will work. Serialization of a cancellation token would not be enough.