Skip to content

Move influxctl --perf-debug example output to command reference#6599

Merged
jstirnaman merged 6 commits intomasterfrom
copilot/sub-pr-6598
Dec 22, 2025
Merged

Move influxctl --perf-debug example output to command reference#6599
jstirnaman merged 6 commits intomasterfrom
copilot/sub-pr-6598

Conversation

Copy link
Contributor

Copilot AI commented Dec 9, 2025

  • Review current state and understand the request
  • Add --perf-debug flag documentation to Cloud Dedicated query.md
  • Add --perf-debug flag documentation to Clustered query.md
  • Add example section showing --perf-debug output in both formats
  • Update release notes to remove example output
  • Test documentation builds correctly
  • Reply to comment
  • Add link to query command in release notes
  • Replace deprecated code-placeholders shortcode with new placeholder attribute syntax
  • Reply to review comments
  • Add disclaimer for example output to indicate it's similar but not exact match
  • Reply to feedback about example output mismatch

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Dec 9, 2025
2 tasks
Copilot AI changed the title [WIP] Release influxctl version 2.12.0 Move influxctl --perf-debug example output to command reference Dec 9, 2025
Copilot AI requested a review from jstirnaman December 9, 2025 16:51
Copy link
Contributor

@jstirnaman jstirnaman left a comment

Choose a reason for hiding this comment

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

Review: Example output doesn't match the documented command

The example performance statistics output was copied from the release notes, but was generated from a different query:

Source Query
Release notes SELECT SUM(i), non_negative_difference(SUM(i)) as diff_i FROM data WHERE time > '2025-11-07T01:20:00Z'...
Documentation SELECT SUM(temp) FROM home WHERE time > now() - 1h GROUP BY time(5m)

The output values (20 rows, 1.222s duration, 70 KiB memory) wouldn't match if a user ran the simplified query.

Action required: Either:

  1. Regenerate the example output using the documented query, or
  2. Preface the output with "The output is similar to the following." if exact reproduction isn't feasible

See DOCS-CONTRIBUTING.md - Code examples for the documentation standard.

Copy link
Contributor Author

Copilot AI commented Dec 19, 2025

Review: Example output doesn't match the documented command

The example performance statistics output was copied from the release notes, but was generated from a different query:...

Added the disclaimer "The output is similar to the following:" before the example output sections in both Cloud Dedicated and Clustered query.md files. This indicates that the output values are representative but may not exactly match when running the simplified query. Changes in commit 3c4c22e.

Copilot AI requested a review from jstirnaman December 19, 2025 22:39
@jstirnaman jstirnaman marked this pull request as ready for review December 19, 2025 23:32
Copilot AI and others added 5 commits December 22, 2025 12:05
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
…d query

Regenerate example performance statistics output using the documented
query (SELECT SUM(temp) FROM home). Tested influxctl query --perf-debug
command to verify output format and values.
@github-actions
Copy link
Contributor

github-actions bot commented Dec 22, 2025

PR Preview Action v1.4.8
🚀 Deployed preview to https://influxdata.github.io/docs-v2/pr-preview/pr-6599/
on branch gh-pages at 2025-12-22 18:56 UTC

@jstirnaman jstirnaman changed the base branch from influxctl-v2.12.0 to master December 22, 2025 18:53
@jstirnaman jstirnaman merged commit 77e52a5 into master Dec 22, 2025
5 checks passed
@jstirnaman jstirnaman deleted the copilot/sub-pr-6598 branch December 22, 2025 19:11
github-actions bot added a commit that referenced this pull request Dec 22, 2025
jstirnaman added a commit that referenced this pull request Dec 22, 2025
* Initial plan

* docs: move perf-debug example output to query command reference

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* docs: update placeholder syntax and add link to query command

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* docs: add disclaimer for example performance statistics output

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* fix(influxdb3): update --perf-debug example output to match documented query

Regenerate example performance statistics output using the documented
query (SELECT SUM(temp) FROM home). Tested influxctl query --perf-debug
command to verify output format and values.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>
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.

2 participants