Skip to content

Update "Find slow queries" to reference query logs instead of audit logs#7187

Merged
leemthompo merged 9 commits into
mainfrom
copilot/update-find-slow-queries
Jul 7, 2026
Merged

Update "Find slow queries" to reference query logs instead of audit logs#7187
leemthompo merged 9 commits into
mainfrom
copilot/update-find-slow-queries

Conversation

Copilot AI commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

The "Find slow queries" section pointed users to audit logs to determine query source. Since 9.4, query logs do this properly: one config, end-to-end request duration, every query type. So the section now leads with that.

leem and Claude cleaned up after Copilot's initial flailing

Copilot AI linked an issue Jul 2, 2026 that may be closed by this pull request
1 task
Copilot AI changed the title [WIP] Update Find slow queries section to link to query logs Update "Find slow queries" to reference query logs instead of audit logs Jul 2, 2026
Copilot AI requested a review from leemthompo July 2, 2026 15:24
@leemthompo

Copy link
Copy Markdown
Member

@copilot we should version tags to keep the information clearly valid for users on different 9.x versions

introruce the new information on a new line and use an inline applies_to tag to make it clear that this is the recommendation for 9.5+

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

@copilot we should version tags to keep the information clearly valid for users on different 9.x versions

introruce the new information on a new line and use an inline applies_to tag to make it cle...

Done in commit Add inline applies_to version tag for query logs recommendation (9.5+). I restored the audit logging info for older versions and added the query logs recommendation on a new line with an inline {applies_to} tag scoped to stack: ga 9.5.

@leemthompo

Copy link
Copy Markdown
Member

@copilot put query logging first instead of the current approach which introduces the old approach first and then forces you to read through that and then says "For a more streamlined approach"

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

@copilot put query logging first instead of the current approach which introduces the old approach first and then forces you to read through that and then says "For a more streamlined approach"

Reordered in the latest commit — query logging now comes first with the {applies_to} tag, followed by the audit logging approach introduced as a fallback "for earlier versions".

Comment thread troubleshoot/elasticsearch/troubleshooting-searches.md Outdated
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

Comment thread troubleshoot/elasticsearch/troubleshooting-searches.md Outdated
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 1 suggestion found

💡 Suggestions (1): Optional style improvements. Apply when helpful.
File Line Rule Message
troubleshoot/elasticsearch/troubleshooting-searches.md 241 Elastic.WordChoice Consider using 'deactivate, deselect, hide, turn off' instead of 'disable', 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.

Frame slow logs as the starting point, then present query logging
(9.4+) and audit logging as version-dependent ways to find the query
source.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@leemthompo leemthompo marked this pull request as ready for review July 3, 2026 11:16
@leemthompo leemthompo requested a review from a team as a code owner July 3, 2026 11:16
Comment thread troubleshoot/elasticsearch/troubleshooting-searches.md Outdated
Co-authored-by: Liam Thompson <leemthompo@gmail.com>

@yetanothertw yetanothertw 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, thank you

@leemthompo leemthompo enabled auto-merge (squash) July 7, 2026 08:49
@leemthompo leemthompo merged commit 9e88599 into main Jul 7, 2026
7 checks passed
@leemthompo leemthompo deleted the copilot/update-find-slow-queries branch July 7, 2026 08:50
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.

[Website]: Find slow queries

3 participants