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:
parent
0494dced3a
commit
51e2813ae4
1 changed files with 2 additions and 3 deletions
|
@ -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
|
||||||
)
|
)
|
||||||
|
|
Loading…
Reference in a new issue