Skip to content

Params: PGSCV_COLLECT_TOP_TABLE, PGSCV_COLLECT_TOP_INDEX, PGSCV_COLLECT_TOP_QUERY#31

Merged
cherts merged 6 commits intocherts:masterfrom
dbulashev:top-k
Aug 8, 2024
Merged

Params: PGSCV_COLLECT_TOP_TABLE, PGSCV_COLLECT_TOP_INDEX, PGSCV_COLLECT_TOP_QUERY#31
cherts merged 6 commits intocherts:masterfrom
dbulashev:top-k

Conversation

@dbulashev
Copy link
Copy Markdown

No description provided.

@cherts
Copy link
Copy Markdown
Owner

cherts commented Jul 29, 2024

@dbulashev Hi, this is a good functionality, but the tests are broken

internal/collector/postgres_statements_test.go:175:73: not enough arguments in call to selectStatementsQuery

need to fix them and add tests for the new functionality.

Thanks!

@cherts
Copy link
Copy Markdown
Owner

cherts commented Jul 29, 2024

I also think it is necessary to keep the compact style of SQL queries in the variables userIndexesQuery, userIndexesQueryTopK and others that have been changed and new ones added.

Thanks!

Comment thread internal/collector/postgres_tables.go Outdated
nullif(sum(coalesce(size_bytes,0)),0),
nullif(sum(coalesce(reltuples,0)),0)
from stat
where not visible and false
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

and false - forgot debugging predicate

@cherts cherts merged commit 2d625f5 into cherts:master Aug 8, 2024
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.

2 participants