plugin::mysql::database - remove default 'UNSET' values

[juniorsysadmin]

Use better types to make it clearer that these parameter are required

[ph448]

@juniorsysadmin what would you consider a 'better type' for the username? IMO the best course of action is to not provide a default and make the manifest fail early and hard with the familiar message saying a String was expected.
scratching a personal itch I have a PR cooking that makes supplying the password optional and sets the default hostname to 'localhost', as this better matches the expectations a MySQL user might have, and it would be nice to complete a fix for this issue.

[juniorsysadmin]

IMO the best course of action is to not provide a default and make the manifest fail early and hard with the familiar message saying a String was expected.

@ph448 Sounds reasonable. I have not touched Puppet for years though.