Skip to content

Conversation

@ncabatoff
Copy link

Only the last column in each metric was being used because the map was being overwritten each time. Also added a dumpmaps option for debugging to help demonstrate the problem.

marcusmartins and others added 9 commits May 5, 2016 18:07
When querying postgres `pg_runtime_variable`, postgres might return -1
for `max_standby_archive_delay` and `max_standby_streaming_delay`
This simply pulls and runs postgres against a number of versions and checks
that we can successfully connect to it.
…_handling

Handle `pg_runtime_variable` return values
…eing used because the map was being overwritten each time); add -dumpmaps option for debugging
@noushi
Copy link
Owner

noushi commented Sep 9, 2016

Excellent!
That feature was the minimal code to get the replication lag metric, I didn't have the time to make it generic enough.
@ncabatoff , thank you!

@noushi noushi merged commit 298b142 into noushi:feature-queries_from_file Sep 9, 2016
@ncabatoff
Copy link
Author

No, thank you! You did 99% of the work, I just fixed a bug.

@noushi
Copy link
Owner

noushi commented Sep 9, 2016

^_^

@ncabatoff ncabatoff deleted the feature-queries_from_file branch September 10, 2016 15:15
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.

5 participants