PEDSnet / pedsnetdcc

CLI tool for PEDSnet data coordinating center ETL tasks
Other
0 stars 0 forks source link

Modify prepdb to allow updating of privileges #69

Open murphyke opened 8 years ago

murphyke commented 8 years ago

pedsnetdcc prepdb can be modified slightly to allow its use for updating privileges. Currently the prepare_database function takes an update keyword argument which is not used by the CLI. The update argument causes database creation to be skipped.

For consistency with the other commands, it would be better to rename update to force, use a custom check_stmt_err to honor the value of force, and add a --force switch to the prepdb CLI command.

Alternatively, we could add a separate 'update_privileges' command, which might be more sensible to people.