Update default.ini #71

Merged
merged 2 commits into from Oct 12, 2011

Conversation

Projects
None yet
3 participants
Owner

davisp commented Oct 12, 2011

Document new options, remove obsolete ones.

@davisp davisp Document custom HTTP/socket options in default.ini
Part of the backport of COUCHDB-1008 to chttpd. This just updates
the default.ini with some commented out options describing their
availability to users.
8a3e76c
Owner

kocolosk commented Oct 12, 2011

If we continue to support the dedicated [chttpd] backlog parameter we should remove backlog from the example config, as the current implementation in chttpd will always ignore it.

Owner

davisp commented Oct 12, 2011

I wasn't entirely certain on the best approach. In a perfect world I'd like to drop backlog but I wouldn't want people to get confused. Perhaps a warning/error if it's set and then document it in the release notes?

Owner

rnewson commented Oct 12, 2011

+1 on removing [chttpd] backlog. You could write a one-liner explaining it moved to the options section immediately below it.

Owner

kocolosk commented Oct 12, 2011

Fine by me if we remove backlog from the top-level chttpd options.

Owner

kocolosk commented Oct 12, 2011

Ok, I removed [chttpd] backlog, [httpd] max_connections (it's a noop in the current mochiweb), and replaced os_process_limit with os_process_soft_limit in [query_server_config (we use a different process manager than vanilla CouchDB). I'd like to clean it further some day but I don't want to hold up the release candidate for it.

Owner

kocolosk commented Oct 12, 2011

I also removed the duplicate documentation of server_options and socket_options in [httpd]. Anyone tweaking those settings for the backend interface is Doing It Wrong.

kocolosk merged commit 82de8a4 into master Oct 12, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment