Skip to content

[refactor] Dynamically concatenate app parameter on the front end to accommodate Prometheus apps#4000

Merged
zqr10159 merged 3 commits into
apache:masterfrom
Saramanda9988:issue_3983
Jan 24, 2026
Merged

[refactor] Dynamically concatenate app parameter on the front end to accommodate Prometheus apps#4000
zqr10159 merged 3 commits into
apache:masterfrom
Saramanda9988:issue_3983

Conversation

@Saramanda9988

@Saramanda9988 Saramanda9988 commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

What's changed?

please refer to #3983

In the frontend, dynamically determine the app parameters of the Prometheus application for related queries in tsdbs

The prefix judgment for the Prometheus in VM has been modified

This is a simplified fix as suggested by @Duansg in #3994

VictoriaMetrics

image

InfluxDb

4c6567dbe15c8bc4951e1848da243036

IoTDb

ce9e78e3ee279498e624e86e4a55acc2

GreptimeDb

image

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

Duansg
Duansg previously approved these changes Jan 23, 2026
@Duansg Duansg requested a review from zqr10159 January 23, 2026 18:59
@yuluo-yx

Copy link
Copy Markdown
Member

need fix conflicts

# Conflicts:
#	hertzbeat-warehouse/src/main/java/org/apache/hertzbeat/warehouse/store/history/tsdb/vm/VictoriaMetricsClusterDataStorage.java
#	hertzbeat-warehouse/src/main/java/org/apache/hertzbeat/warehouse/store/history/tsdb/vm/VictoriaMetricsDataStorage.java
@Saramanda9988

Copy link
Copy Markdown
Contributor Author

need fix conflicts

Done

@zqr10159 zqr10159 left a comment

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.

LGTM

@zqr10159 zqr10159 merged commit 634006a into apache:master Jan 24, 2026
4 checks passed
@Saramanda9988 Saramanda9988 deleted the issue_3983 branch January 24, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants