Re: Proposal - GroupCommitLogService

2017-05-14 Thread Hiroyuki Yamada
Hi all, > Yuji Great work ! That's what I expected for batch. (I was the one who wrote the jira below. ) I think it might be better for us to have your proposed group-commit feature as "batch" instead of having another commit-log-service. Th

Re: Proposal - GroupCommitLogService

2017-05-13 Thread Yuji Ito
That's exactly the motivation of this proposal. Batch can group only writes which are not persisted (not kept waiting) at that time. In Batch, a write of commitlog isn't kept waiting because the thread lock (the semaphore in 2.2 and 3.0) for sync is released immediately. So, the window size means

Re: Proposal - GroupCommitLogService

2017-05-13 Thread Jonathan Ellis
Does that mean that Batch is not working as designed? If there are other pending writes, Batch should also group them together. (Did you test with giving Batch the same window size as Group?) On Sat, May 13, 2017 at 10:08 AM, Yuji Ito wrote: > Batch outperforms when there is no concurrency. >

Re: Proposal - GroupCommitLogService

2017-05-13 Thread Yuji Ito
Batch outperforms when there is no concurrency. Because GroupCommit should wait the window time, the throughput and the latency are worse in a single request. GroupCommit can gather the commitlog writes which are requested in the window time. Actually, the throughput of a single thread was bounded

Re: Proposal - GroupCommitLogService

2017-05-13 Thread Jonathan Ellis
Can we replace Batch entirely with this, or are there situations where Batch would outperform (in latency, for instance)? On Sat, May 13, 2017 at 7:21 AM, Yuji Ito wrote: > Hi dev, > > I propose a new CommitLogService, GroupCommitLogService, to improve the > throughput when lots of requests are

Re: Proposal - GroupCommitLogService

2017-05-13 Thread Yuji Ito
Thanks Jeremiah, I've opened a ticket on JIRA. https://issues.apache.org/jira/browse/CASSANDRA-13530 Best, Yuji On Sat, May 13, 2017 at 9:38 PM, J. D. Jordan wrote: > Sounds interesting. You should open a JIRA and attach your code for > discussion of it. > > https://issues.apache.org/jira/br

Re: Proposal - GroupCommitLogService

2017-05-13 Thread J. D. Jordan
Sounds interesting. You should open a JIRA and attach your code for discussion of it. https://issues.apache.org/jira/browse/CASSANDRA/ -Jeremiah > On May 13, 2017, at 7:21 AM, Yuji Ito wrote: > > Hi dev, > > I propose a new CommitLogService, GroupCommitLogService, to improve the > throughpu

Proposal - GroupCommitLogService

2017-05-13 Thread Yuji Ito
Hi dev, I propose a new CommitLogService, GroupCommitLogService, to improve the throughput when lots of requests are received. It improved the throughput by maximum 94%. I'd like to discuss about this CommitLogService. Currently, we can select either 2 CommitLog services; Periodic and Batch. In P