Juliusz Stasiewicz

> ``` > if (handler.is_counter() && coordinator == my_address) { > + ++stats.writes_attempts.get_ep_stat(coordinator); > got_response(response_id, coordinator, std::nullopt); > } else { > if (!handler.read_repair_write()) { > ``` I was thinking...

> What does it matter if you do not count failures. It is even more confusing. It explicitly says "attempts". What would be confusing IMHO is not* accounting failures here....

@avelanarius IIRC the reason was to separate schema extensions from regular table options. In general, putting schema extensions among "normal" options, like `AND schema_extension_X = abc`, can lead to an...

For future readers: it also might be worth noting that: - we used shard-aware drivers - `scylla_storage_proxy_replica_cross_shard_ops` metrics don't indicate any inter-shard operations - the behavior is observable even for...