Skip to content
This repository was archived by the owner on Oct 17, 2022. It is now read-only.

Document seq_interval parameter#261

Merged
wohali merged 2 commits intoapache:masterfrom
jjrodrig:230-seq_interval
Mar 18, 2018
Merged

Document seq_interval parameter#261
wohali merged 2 commits intoapache:masterfrom
jjrodrig:230-seq_interval

Conversation

@jjrodrig
Copy link
Copy Markdown
Contributor

Overview

Added the seq_interval parameter description as suggested by @wohali in #230 with minor changes.
A note has been added referencing the improvement gained by other projects when using this parameter

Testing recommendations

GitHub issue number

This PR fixes #230

Related Pull Requests

Checklist

  • Documentation is written and is accurate;
  • make check passes with no errors
  • Update rebar.config.script with the commit hash once this PR is rebased and merged

.. note::
Cloudant Sync and PouchDB already optimize the replication process by
setting ``seq_interval`` parameter to the number of results expected per
batch. This parameter increase throughput by reducing latency between
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/increase/increases/

@wohali
Copy link
Copy Markdown
Member

wohali commented Mar 18, 2018

+1, but for future reference, note PouchDB may not use seq_interval because of an open bug.

See: apache/couchdb#1221 .

@wohali wohali merged commit 87a4f3f into apache:master Mar 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

seq_interval parameter for /db/_changes endpoint is not documented

3 participants