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

Start of PostgreSQL module #2253

Merged
merged 1 commit into from
Aug 12, 2016
Merged

Start of PostgreSQL module #2253

merged 1 commit into from
Aug 12, 2016

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Aug 12, 2016

A start for the Metricbeat Postgresql module.

The activity metricset (with data about the currently running processes) is done.

@tsg tsg added in progress Pull request is currently in progress. review Metricbeat Metricbeat labels Aug 12, 2016
- module: postgresql
metricsets: ["activity"]
enabled: true
period: 1s
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we set the default here also to 10s like the other metricsets?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's one of the the TODOs above :)

@ruflin
Copy link
Member

ruflin commented Aug 12, 2016

LGTM

@tsg tsg changed the title PostgreSQL module Start of PostgreSQL module Aug 12, 2016
@tsg tsg mentioned this pull request Aug 12, 2016
6 tasks
@tsg
Copy link
Contributor Author

tsg commented Aug 12, 2016

@ruflin I addressed the comments and moved the TODOs in a meta ticket: #2255.

@tsg tsg removed the in progress Pull request is currently in progress. label Aug 12, 2016
}

// Fetch methods implements the data gathering and data conversion to the right format
// It returns the event which is then forward to the output. In case of an error, a
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could also remove the generator comments here and above for init and New.

@tsg tsg force-pushed the metricbeat_postgres branch 2 times, most recently from da73af5 to 6abea48 Compare August 12, 2016 13:30
A start for the Metricbeat Postgresql module. The core functionality for connecting and an `activity` metricset is done.
@ruflin ruflin merged commit 9cdd06e into elastic:master Aug 12, 2016
@tsg tsg deleted the metricbeat_postgres branch August 25, 2016 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants