Skip to content

cannot parse some SHOW variables #15

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

Closed
jinty opened this issue Jul 27, 2016 · 1 comment · Fixed by #17
Closed

cannot parse some SHOW variables #15

jinty opened this issue Jul 27, 2016 · 1 comment · Fixed by #17

Comments

@jinty
Copy link
Contributor

jinty commented Jul 27, 2016

I was trying out the latest master and get this:

time="2016-07-27T14:07:50Z" level=error msg="Unexpected error parsing column:  pg max_pred_locks_per_transaction 64" file="postgres_exporter.go" line=471
time="2016-07-27T14:07:50Z" level=error msg="Unexpected error parsing column:  pg max_identifier_length 63" file="postgres_exporter.go" line=471
time="2016-07-27T14:07:50Z" level=error msg="Unexpected error parsing column:  pg max_locks_per_transaction 64" file="postgres_exporter.go" line=471

This is querying postgresql 9.5. It seems that the query:

SHOW max_pred_locks_per_transaction;

returns a "text" type. But eyeballing the code, that should work...

@wrouesnel
Copy link
Contributor

Interesting, AFAIK all of those types of queries should be working fine on 9.5 (there are harmless errors only when stepping back in versions).

jinty pushed a commit to jinty/postgres_exporter that referenced this issue Jul 28, 2016
ritbl pushed a commit to heniek/postgres_exporter that referenced this issue Mar 19, 2023
dehaansa pushed a commit to dehaansa/postgres_exporter that referenced this issue Oct 24, 2024
* lint fixes

* Add .lint exclusion file - for mixtool

* Update .lint exclusion file - for mixtool

* reverting overview dashboard changes

* delete postgres-overview.libsonnet

* bringing back Vitaly's changes

* updating instance template definition, adding job filter to datname template
Sticksman pushed a commit to Sticksman/postgres_exporter that referenced this issue Apr 24, 2025
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 a pull request may close this issue.

2 participants