Skip to content

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

Merged
glinton merged 1 commit intoinfluxdata:masterfrom
axinojolais:fix_postgres_ext_v10
Oct 30, 2018
Merged

postgresql_extensible: fix DB version check for version 10+#4866
glinton merged 1 commit intoinfluxdata:masterfrom
axinojolais:fix_postgres_ext_v10

Conversation

@axinojolais
Copy link
Contributor

@axinojolais axinojolais commented Oct 16, 2018

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

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
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants