Skip to content

[microsoft_sqlserver] add new field buffer_cache_hit_ratio_pct - #18874

Merged
jakubgalecki0 merged 5 commits into
elastic:mainfrom
jakubgalecki0:mssql_buffer_hit_rate
May 8, 2026
Merged

[microsoft_sqlserver] add new field buffer_cache_hit_ratio_pct#18874
jakubgalecki0 merged 5 commits into
elastic:mainfrom
jakubgalecki0:mssql_buffer_hit_rate

Conversation

@jakubgalecki0

@jakubgalecki0 jakubgalecki0 commented May 7, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

Introduce new buffer_cache_hit_ratio_pct that calculates buffer hit cache ratio by calculating Buffer cache hit ratio / Buffer cache hit ratio base.

Tested on kibana versions: 8.19.0, 9.2.1, 9.4.0

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

image image
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

Vale Linting Results

Summary: 4 warnings, 1 suggestion found

⚠️ Warnings (4)
File Line Rule Message
packages/microsoft_sqlserver/docs/README.md 491 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'and so on' instead of 'etc'.
packages/microsoft_sqlserver/docs/README.md 491 Elastic.DirectionalLanguage Don't use directional language. Use 'earlier on this page' instead of 'noted above'.
packages/microsoft_sqlserver/docs/README.md 492 Elastic.DirectionalLanguage Don't use directional language. Use 'earlier on this page' instead of 'noted above'.
packages/microsoft_sqlserver/docs/README.md 493 Elastic.QuotesPunctuation Place punctuation inside closing quotation marks.
💡 Suggestions (1)
File Line Rule Message
packages/microsoft_sqlserver/docs/README.md 501 Elastic.WordChoice Consider using 'select, press, visits' instead of 'hit', unless the term is in the UI.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

Package microsoft_sqlserver 👍(2) 💚(0) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
audit 2386.63 1893.94 -492.69 (-20.64%) 💔

To see the full report comment with /test benchmark fullreport

@jakubgalecki0
jakubgalecki0 requested a review from muthu-mps May 7, 2026 14:47
changes:
- description: Add `mssql.metrics.buffer_cache_hit_ratio_pct` field to the `performance` data stream
type: enhancement
link: https://github.com/elastic/integrations/pull/1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fix PR link,

Suggested change
link: https://github.com/elastic/integrations/pull/1
link: https://github.com/elastic/integrations/pull/18874
@muthu-mps

Copy link
Copy Markdown
Contributor

@jakubgalecki0 - Can we update the latest dashboard image?

@jakubgalecki0

Copy link
Copy Markdown
Contributor Author

@muthu-mps updated

@jakubgalecki0
jakubgalecki0 marked this pull request as ready for review May 8, 2026 07:18
@jakubgalecki0
jakubgalecki0 requested a review from a team as a code owner May 8, 2026 07:18
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

@muthu-mps muthu-mps left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@jakubgalecki0
jakubgalecki0 merged commit 9a02a86 into elastic:main May 8, 2026
11 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package microsoft_sqlserver - 2.17.0 containing this change is available at https://epr.elastic.co/package/microsoft_sqlserver/2.17.0/

@andrewkroh andrewkroh added dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:microsoft_sqlserver Microsoft SQL Server labels May 11, 2026
herrBez pushed a commit to herrBez/integrations that referenced this pull request Jun 1, 2026
…ic#18874)

* [microsoft_sqlserver] add new field buffer_cache_hit_ratio_pct

* add new panel

* update screenshot and pr link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:microsoft_sqlserver Microsoft SQL Server

4 participants