Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for oplog file-delete logging issue with out-of-scope variables #38

Merged
merged 1 commit into from
Jul 22, 2016
Merged

Fix for oplog file-delete logging issue with out-of-scope variables #38

merged 1 commit into from
Jul 22, 2016

Conversation

timvaillancourt
Copy link
Contributor

Fix for #37, which repeated the same host/port in log lines due to using out-of-scope variables. Added host/port to a dict in the oplog_files array so we can print it.

BEFORE:

[2016-07-22 16:09:58,863] [DEBUG] [MainProcess] [Resolver:run:89] Deleting tailed oplog file for localhost:27017
[2016-07-22 16:09:58,863] [DEBUG] [MainProcess] [Resolver:run:89] Deleting tailed oplog file for localhost:27017
[2016-07-22 16:09:58,863] [DEBUG] [MainProcess] [Resolver:run:89] Deleting tailed oplog file for localhost:27017
[2016-07-22 16:09:58,863] [DEBUG] [MainProcess] [Resolver:run:89] Deleting tailed oplog file for localhost:27017

AFTER:

[2016-07-22 16:08:29,208] [DEBUG] [MainProcess] [Resolver:run:93] Deleting tailed oplog file for centos7:27029
[2016-07-22 16:08:29,208] [DEBUG] [MainProcess] [Resolver:run:93] Deleting tailed oplog file for localhost:29017
[2016-07-22 16:08:29,208] [DEBUG] [MainProcess] [Resolver:run:93] Deleting tailed oplog file for localhost:28027
[2016-07-22 16:08:29,209] [DEBUG] [MainProcess] [Resolver:run:93] Deleting tailed oplog file for localhost:27017

@dbmurphy dbmurphy merged commit 3b474ab into Percona-Lab:master Jul 22, 2016
@timvaillancourt timvaillancourt deleted the issue_37_var_outofscope branch July 22, 2016 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants