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.

Sign up for GitHub

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

Jump to bottom

postgresql_extensible: fix DB version check for version 10+ #4866

Merged
merged 1 commit into from
Oct 30, 2018
Merged

postgresql_extensible: fix DB version check for version 10+ #4866

merged 1 commit into from
Oct 30, 2018

Conversation

axinojolais
Copy link
Contributor

@axinojolais axinojolais commented Oct 16, 2018 •
edited
Loading

Fixes #3864
Idea from @robinsmidsrod
#3864 (comment)

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

No need for README update.
Unsure how to emulate PG version in unit tests

glinton
glinton approved these changes Oct 17, 2018
View reviewed changes
Copy link
Contributor

@glinton glinton left a comment

Choose a reason for hiding this comment

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

Thanks

@glinton glinton added feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin hacktoberfest labels Oct 17, 2018
@danielnelson danielnelson added this to the 1.8.3 milestone Oct 17, 2018
@danielnelson danielnelson self-requested a review October 17, 2018 18:45
@glinton glinton merged commit 69d21a5 into influxdata:master Oct 30, 2018
glinton pushed a commit that referenced this pull request Oct 30, 2018
@axinojolais @glinton
Fix version check in postgresql_extensible (#4866)
20529cc 
(cherry picked from commit 69d21a5)
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
@axinojolais @otherpirate
Fix version check in postgresql_extensible (influxdata#4866)
5a5cf2a
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
@axinojolais @otherpirate
Fix version check in postgresql_extensible (influxdata#4866)
2aff52f
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
@axinojolais
Fix version check in postgresql_extensible (influxdata#4866)
3c19964
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
@axinojolais
Fix version check in postgresql_extensible (influxdata#4866)
6d77f02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielnelson danielnelson danielnelson approved these changes

@glinton glinton glinton approved these changes

Assignees
No one assigned
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Milestone
1.8.3
Development

Successfully merging this pull request may close these issues.
3 participants
@axinojolais @danielnelson @glinton