Skip to content

Add schema table with explicit data types for query_log fields#6868

Merged
jstirnaman merged 2 commits intoritwika/query_logfrom
copilot/update-schema-data-type-table
Feb 25, 2026
Merged

Add schema table with explicit data types for query_log fields#6868
jstirnaman merged 2 commits intoritwika/query_logfrom
copilot/update-schema-data-type-table

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

Replaced categorized bullet list with structured schema table for the query_log table's 21 fields.

Changes

  • Schema documentation: Bulleted prose → 3-column table (Column | Data type | Description)
  • Type coverage: Added explicit data types for all fields (timestamp, string, int64)
  • Completeness: Enumerated all fields individually vs. grouped categories with vague "and ingester-related fields"

Before

- **Timing**: `time` (timestamp), `query_issue_time_ns`, `permit_duration_ns`, ...
- **Resource usage**: `partition_count`, `parquet_file_count`, `max_memory_bytes`, and ingester-related fields.

After

| Column | Data type | Description |
| :----- | :-------- | :---------- |
| **time** | timestamp | Timestamp when the query log entry was recorded |
| **query_issue_time_ns** | int64 | Time when the query was issued (nanoseconds) |
| **permit_duration_ns** | int64 | Time spent waiting for query permit (nanoseconds) |
| **partition_count** | int64 | Number of partitions accessed |
| **parquet_file_count** | int64 | Number of Parquet files read |
| **max_memory_bytes** | int64 | Maximum memory used during query execution (bytes) |

Impact

  • Developer UX: Quick reference for SQL query construction
  • LLM consumption: Structured data for markdown generation pipeline (TurndownService preserves tables)
  • Tooling: Machine-readable schema for SDK/type generation

Preserves existing note about success/running being string-typed ('true'/'false'), not boolean.

Original prompt

Pull request: https://github.com/influxdata/docs-v2/pull/6866/files

At line 131, the section describes fields. Should we include a schema or data type table for clarity?


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…nd descriptions

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Copilot AI changed the title [WIP] Suggest including schema and data type table for clarity Add schema table for query_log fields with data types Feb 25, 2026
Copilot AI requested a review from jstirnaman February 25, 2026 14:00
@jstirnaman jstirnaman marked this pull request as ready for review February 25, 2026 14:28
@jstirnaman jstirnaman requested a review from a team as a code owner February 25, 2026 14:28
@jstirnaman jstirnaman requested review from ritwika314 and sanderson and removed request for a team February 25, 2026 14:28
Copilot AI changed the title Add schema table for query_log fields with data types Add schema table with explicit data types for query_log fields Feb 25, 2026
Copilot AI requested a review from jstirnaman February 25, 2026 17:19
Copy link
Collaborator

@ritwika314 ritwika314 left a comment

Choose a reason for hiding this comment

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

lgtm, Thank you!

@jstirnaman jstirnaman merged commit e8e5607 into ritwika/query_log Feb 25, 2026
7 checks passed
@jstirnaman jstirnaman deleted the copilot/update-schema-data-type-table branch February 25, 2026 22:03
jstirnaman added a commit that referenced this pull request Feb 25, 2026
* Add documentation for viewing the query log in the Admin UI and using influxctl

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Replace deprecated code-placeholders shortcode with placeholders attribute (#6869)

* Initial plan

* Replace deprecated code-placeholders shortcode with placeholders attribute

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

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Fix support.influxdata.com link pattern in query-log.md (#6867)

* Initial plan

* Fix support link pattern and add enforcement check

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

* Remove validation changes, keep content fix only

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

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>

* Apply suggestions from code review

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>

* Clarify note on query log enabling and its impact on cluster load per @jeffreyssmith2nd 's suggestion

* Add schema table with explicit data types for query_log fields (#6868)

* Initial plan

* Add comprehensive schema table for query_log fields with data types and descriptions

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

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

---------

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
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.

3 participants