Skip to content

docs(influxql): restore ordering caution for FUTURE/PAST LIMIT on versions prior to v1.12.3#6944

Closed
Copilot wants to merge 2 commits intomasterfrom
copilot/restore-ordering-warning-influxql
Closed

docs(influxql): restore ordering caution for FUTURE/PAST LIMIT on versions prior to v1.12.3#6944
Copilot wants to merge 2 commits intomasterfrom
copilot/restore-ordering-warning-influxql

Conversation

Copy link
Contributor

Copilot AI commented Mar 14, 2026

PR #6872 corrected the FUTURE LIMIT/PAST LIMIT clause order in the docs to match the v1.12.3 fix, but left no warning for users still on older versions where the reversed order (PAST LIMIT before FUTURE LIMIT) is required.

Changes

  • content/influxdb/v1/query_language/spec.md — Adds [!Caution] after the existing ordering [!Note] in both CREATE DATABASE and CREATE RETENTION POLICY grammar sections, directing users on pre-v1.12.3 to use the old order and linking to the v1.12.3 release notes.

  • content/enterprise_influxdb/v1/query_language/spec.md — Same cautions, but without an upgrade suggestion since Enterprise v1.12.3 is not yet GA.

Each affected grammar block now reads:

> [!Note]
> When using both `FUTURE LIMIT` and `PAST LIMIT` clauses, `FUTURE LIMIT` must appear before `PAST LIMIT`.

> [!Caution]
> **Prior to InfluxDB v1.12.3**, when using both `FUTURE LIMIT` and `PAST LIMIT` clauses,
> `PAST LIMIT` must appear before `FUTURE LIMIT`.
> [Upgrade to InfluxDB v1.12.3 or later] to use the order shown above.
Original prompt

prior to #6872, we had a warning about the ordering of future and past limits in InfluxQL for v1. Restore it as a caution callout for users not yet on v1.12.3. The caution for Enterprise shouldn't suggest upgrading--the release isn't generally available yet.


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

…1.12.3

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Copilot AI changed the title [WIP] Add caution callout about InfluxQL future and past limits docs(influxql): restore ordering caution for FUTURE/PAST LIMIT on versions prior to v1.12.3 Mar 14, 2026
Copilot AI requested a review from jstirnaman March 14, 2026 11:41
@jstirnaman jstirnaman added release/v1 InfluxDB v1 OSS/Enterprise release documentation product:v1 InfluxDB v1 OSS product:v1-enterprise InfluxDB Enterprise v1 labels Mar 14, 2026
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.

I'll merge this into the v1.12.3 redo (delaying Ent publish until GA) and update it there.

@jstirnaman
Copy link
Contributor

Closing — the caution notes were incorrect. The correct InfluxQL syntax has always been FUTURE LIMIT before PAST LIMIT. The v1.12.3 fix addresses Enterprise meta-node recording, not parser syntax order. See #6945 review comments.

@jstirnaman jstirnaman closed this Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:v1-enterprise InfluxDB Enterprise v1 product:v1 InfluxDB v1 OSS release/v1 InfluxDB v1 OSS/Enterprise release documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants