Set --quick to True by default and make it selectable (#3418)

As indicated by @jmainguy in eb1cb94380 (commitcomment-17077470), this will allow the user to set the option quick, which is True by default (as it was before).

This is in reference to: https://github.com/ansible/ansible-modules-core/pull/3402
This commit is contained in:
hyperized 2016-04-13 11:15:48 +02:00 committed by René Moser
parent 79ff959635
commit 48fa3a5243

View file

@ -66,7 +66,7 @@ options:
description: description:
- Option used for dumping large tables - Option used for dumping large tables
required: false required: false
default: false default: true
version_added: "2.1" version_added: "2.1"
author: "Ansible Core Team" author: "Ansible Core Team"
requirements: requirements:
@ -122,7 +122,6 @@ def db_dump(module, host, user, password, db_name, target, all_databases, port,
# If defined, mysqldump demands --defaults-extra-file be the first option # If defined, mysqldump demands --defaults-extra-file be the first option
if config_file: if config_file:
cmd += " --defaults-extra-file=%s" % pipes.quote(config_file) cmd += " --defaults-extra-file=%s" % pipes.quote(config_file)
cmd += " --quick"
if user is not None: if user is not None:
cmd += " --user=%s" % pipes.quote(user) cmd += " --user=%s" % pipes.quote(user)
if password is not None: if password is not None:
@ -248,7 +247,7 @@ def main():
connect_timeout=dict(default=30, type='int'), connect_timeout=dict(default=30, type='int'),
config_file=dict(default="~/.my.cnf", type='path'), config_file=dict(default="~/.my.cnf", type='path'),
single_transaction=dict(default=False, type='bool'), single_transaction=dict(default=False, type='bool'),
quick=dict(default=False, type='bool'), quick=dict(default=True, type='bool'),
), ),
supports_check_mode=True supports_check_mode=True
) )