Re: Coordinator Write Metrics per CF

2018-03-13 Thread Sumanth Pasupuleti
I have submitted a patch for https://issues.apache.org/jira/browse/CASSANDRA-14232. Looking for reviewers/feedback. Thanks, Sumanth On Tue, Feb 13, 2018 at 2:06 PM, Benedict Elliott Smith wrote: > Sorry, I guess I'm tired. I thought this was discussing local write >

Re: Coordinator Write Metrics per CF

2018-02-13 Thread Benedict Elliott Smith
Sorry, I guess I'm tired. I thought this was discussing local write latency. I'm surprised we have that and not coordinator write latency. Please do ignore me, I'm not sure why I got involved! On 13 February 2018 at 21:48, Benedict Elliott Smith wrote: > For the record,

Re: Coordinator Write Metrics per CF

2018-02-13 Thread Benedict Elliott Smith
For the record, I'm not certain there *is* a great deal of value in this. The read latency metrics are expected to vary a great deal, since the entire IO subsystem is involved. Writes, however, go straight to a memtable. They only block on IO if we exceed our commit log flush bandwidth for an

Re: Coordinator Write Metrics per CF

2018-02-13 Thread Sumanth Pasupuleti
Thanks Kurt and Chris for your valuable inputs. Created https://issues.apache.org/jira/browse/CASSANDRA-14232; I shall start working on this. Thanks, Sumanth On Mon, Feb 12, 2018 at 7:43 AM, Chris Lohfink wrote: > It would be good to have it. Its not that its not there

Re: Coordinator Write Metrics per CF

2018-02-12 Thread Chris Lohfink
It would be good to have it. Its not that its not there because its difficult or anything. I think its more that the read latency metric was needed for speculative retry so it was added but the write side wasn't needed for that feature so wasn't added at same time. It would be very useful in