[
https://issues.apache.org/jira/browse/COUCHDB-762?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adam Kocoloski updated COUCHDB-762:
-----------------------------------
Attachment: patch-to-reproduce-benchmarks.txt
This is not the patch I would submit, but rather one that exports both new and
old methods for benchmarking.
> Faster implementation of couch_file:pread_iolist
> ------------------------------------------------
>
> Key: COUCHDB-762
> URL: https://issues.apache.org/jira/browse/COUCHDB-762
> Project: CouchDB
> Issue Type: Improvement
> Components: Database Core
> Affects Versions: 0.11
> Environment: any
> Reporter: Adam Kocoloski
> Priority: Minor
> Fix For: 1.1
>
> Attachments: patch-to-reproduce-benchmarks.txt,
> pread_iolist_results.txt
>
>
> couch_file's pread_iolist function is used every time we read anything from
> disk. It makes 2-3 gen_server calls to the couch_file process to do its work.
> This patch moves the work done by the read_raw_iolist function into the
> gen_server itself and adds a pread_iolist handler. This means that one
> gen_server call is sufficient in every case.
> Here are some benchmarks comparing the current method with the patch that
> reduces everything to one call. I write a number of 10k binaries to a file,
> then read them back in a random order from 1/5/10/20 concurrent reader
> processes. I report the median/90/95/99 percentile response times in
> microseconds. In almost every case the patch is an improvement.
> The data was fully cached for these tests; I think that in a real-world
> concurrent reader scenario the performance improvement may be greater. The
> patch ensures that the 2-3 pread calls reading sequential bits of data (term
> length, MD5, and term) are always submitted without interruption.
> Previously, two concurrent readers could race to read different terms and
> cause some extra disk head movement.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
