Skip to content

Conversation

@zqr10159
Copy link
Member

@zqr10159 zqr10159 commented Dec 2, 2025

This pull request introduces several improvements and new features related to log querying and ingestion. The main focus is on enhancing log search capabilities by adding support for keyword-based filtering in log queries and statistics endpoints. Additionally, there are adjustments to logging verbosity and updates to dependencies, including support for PostgreSQL in the warehouse module.

Log Query and Statistics Enhancements:

  • Added a search keyword parameter to log query and statistics endpoints in LogQueryController, enabling users to filter logs by content. All relevant controller methods and internal helper methods now support this new parameter. [1] [2] [3] [4] [5]
  • Updated the internal log filtering and pagination methods to pass and handle the new search parameter, ensuring consistent keyword-based log filtering across all queries.

Logging Verbosity Adjustments:

  • Changed log statements from info to debug level in both LogIngestionController and OtlpLogProtocolAdapter to reduce log noise during normal operation. [1] [2]

Warehouse Module Improvements:

  • Added PostgreSQL connection support in the warehouse configuration (application.yml) and included the PostgreSQL JDBC driver as a dependency in hertzbeat-warehouse/pom.xml. [1] [2]
  • Added spring-boot-starter-jdbc as a provided dependency to support JDBC-based integrations.

Code and Dependency Cleanup:

  • Removed redundant comments from the pom.xml file for better readability and maintainability. [1] [2] [3] [4] [5]

These changes collectively improve log search functionality, optimize logging output, and extend database support for the warehouse module.## What's changed?

zqr10159 added 30 commits April 23, 2024 10:35
Signed-off-by: Yang Chen <1597081640@qq.com>
Copilot AI review requested due to automatic review settings December 3, 2025 03:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@bigcyy bigcyy merged commit 7f4c2e0 into apache:master Dec 3, 2025
4 checks passed
bigcyy added a commit that referenced this pull request Dec 3, 2025
zqr10159 added a commit to zqr10159/hertzbeat that referenced this pull request Dec 3, 2025
Signed-off-by: Yang Chen <1597081640@qq.com>
Co-authored-by: Yang Chen <1597081640@qq.com>
(cherry picked from commit 7f4c2e0)
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.

2 participants