Skip to content

[feat]:add db account monitoring metrics for postgresql.#4131

Open
wy471x wants to merge 7 commits into
apache:masterfrom
wy471x:feat_db-account_monitoring_metrics-#3737
Open

[feat]:add db account monitoring metrics for postgresql.#4131
wy471x wants to merge 7 commits into
apache:masterfrom
wy471x:feat_db-account_monitoring_metrics-#3737

Conversation

@wy471x
Copy link
Copy Markdown

@wy471x wy471x commented Apr 29, 2026

Related Task#3737:

All changes:
1.Add monitoring info to postgresql yaml.
2.Update description markdown docs.

Test case:
截屏2026-04-30 10 13 36
截屏2026-04-30 10 13 14

@github-actions github-actions Bot added doc Improvements or additions to documentation monitoring-template backend labels Apr 29, 2026
@wy471x
Copy link
Copy Markdown
Author

wy471x commented May 7, 2026

@Duansg Hello, I have fixed the CI issue. Please take a look when you have time.

WHEN valuntil < NOW() THEN 'EXPIRED'
ELSE 'OPEN'
END as account_status
FROM pg_user
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Starting with version 8.1, PostgreSQL adopted the roles model. I see that the old users compatibility view is being used here. Should we consider switching to roles?

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.

Okay, I also agree with your opinion. I have made modifications to this. Please take a look when you have time. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend doc Improvements or additions to documentation monitoring-template

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants