Skip to content
This repository was archived by the owner on Feb 18, 2025. It is now read-only.

WIP: The Great Configuration Variables Exodus (deprecation)#150

Merged
shlomi-noach merged 28 commits into
masterfrom
config-variables-exodus
May 21, 2017
Merged

WIP: The Great Configuration Variables Exodus (deprecation)#150
shlomi-noach merged 28 commits into
masterfrom
config-variables-exodus

Conversation

@shlomi-noach
Copy link
Copy Markdown
Collaborator

This (ongoing) PR deprecates configuration variables that are:

  • unused
  • not so important to have configurable (can do with magic values)
  • used to be used for backwards compatibility but should now just go away
  • can be superseded by other solutions

@shlomi-noach
Copy link
Copy Markdown
Collaborator Author

shlomi-noach commented Apr 13, 2017

Listing of deprecated variables (to be edited as PR proceeds)

  • DatabaselessMode__experimental -- with ongoing work to run on sqlite this variable becomes irrelevant.
  • BufferBinlogEvents -- should always be true. It works better and places less locks on the backend server.
  • BinlogFileHistoryDays is now just a magic number.
  • MaintenanceOwner is now a magic string.
  • ReadLongRunningQueries -- it's just not orchestrators job! Removing this feature.
  • DiscoveryPollSeconds was already deprecated. Now a magic number.
  • ActiveNodeExpireSeconds is now a magic number. Since DiscoveryPollSeconds is 1sec, ActiveNodeExpireSeconds can follow suit.
  • AuditPageSize is now a magic number
  • AuditPurgeDays is now a magic number (31)
  • SlaveStartPostWaitMilliseconds now deprecated. Automated, incremental sleep tries to minimize sleep time.
  • MySQLTopologyMaxPoolConnections is now a constant
  • MaintenancePurgeDays is now a constant
  • MaintenanceExpireMinutes is now a constant
  • HttpTimeoutSeconds changed to AgentHttpTimeoutSeconds and is now a constant
  • AgentAutoDiscover is deprecated. Assumed true
  • PseudoGTIDCoordinatesHistoryHeuristicMinutes is now a constant

@shlomi-noach
Copy link
Copy Markdown
Collaborator Author

shlomi-noach commented May 21, 2017

The list is growing and the PR is open for too long. I will resolve + merge this PR and will open another one for more deprecation.

@shlomi-noach shlomi-noach temporarily deployed to production May 21, 2017 05:17 Inactive
@shlomi-noach shlomi-noach merged commit 762b86e into master May 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants