New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Normalize some config options #3609
Conversation
src/couch/src/couch_compress.erl
Outdated
|
||
|
||
get_compression_method() -> | ||
case config:get("couchdb", "file_compression") of | ||
case config:get("couchdb", "file_compression", ?DEFAULT_FILE_COMPRESSION) of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can re-use the constant from couch_db.hrl:
https://github.com/apache/couchdb/blob/3.x/src/couch/include/couch_db.hrl#L16
And I think that would happen if we just leave the code as is. In the line below if config:get("couchdb", "file_compression")
is undefined
it should pick ?DEFAULT_COMPRESSION
which is snappy
(an atom).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed back.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1. Great work.
Overview
Normalize some configuration options to comment out the default.ini file.
Testing recommendations
Related Issues or Pull Requests
fixes #3473
Checklist
rel/overlay/etc/default.ini