Add parameterization changes from wikipedia. #98
Conversation
This commit resolves a few small errors that were introduced in Monty's parameter adding patch. All of these resolved errors were preventing the catalog from being applied correctly. - removes second reference to $tmp_table_size - removes reference in template to undefined variables tmpdir and gid
|
Even with #93, I think this pull request is still viable. It makes many of the defaults tweakable, and is a much cleaner original template. This needs rebasing, and I would like tests for the added parameters, though I don't know if that should be a merge blocker. (We don't have a policy on that.) |
|
@bodepd Are we going down this path or dropping the pull request? |
|
I would defer this to @bodepd. But would endorse it for the previously stated reasons. |
|
I would prefer to drop it in favor of writing native types to manage the ini config file. This way users can add whatever customization they want from top scope. and the config class can just contain reasonable defaults. |
|
@bodepd You'll probably want to expand on that. Do you think you could write a short blurb about how to use the |
|
@bodepd, do you think we ought to still provide the class interfaces and just back them with ini_setting declarations instead of templates? |
(FM-8159) Add Windows Server 2019 support
No description provided.