Skip to content

fix(enterprise): correct FUTURE/PAST LIMIT docs for v1.12.2#6947

Merged
jstirnaman merged 1 commit intoworktree-jts-fix-unpublish-ent-1.12.3from
jts-enterprise-v1.12.2-fixes
Mar 14, 2026
Merged

fix(enterprise): correct FUTURE/PAST LIMIT docs for v1.12.2#6947
jstirnaman merged 1 commit intoworktree-jts-fix-unpublish-ent-1.12.3from
jts-enterprise-v1.12.2-fixes

Conversation

@jstirnaman
Copy link
Contributor

Summary

  • Fixes documentation bugs in Enterprise v1.12.2 InfluxQL docs where PAST LIMIT incorrectly appeared before FUTURE LIMIT
  • The correct syntax has always been FUTURE LIMIT before PAST LIMIT — this is a doc bug, not a parser change
  • Reorders grammar productions in spec.md (CREATE DATABASE, CREATE RETENTION POLICY) to match actual parser behavior
  • Adds ordering note callouts explaining that FUTURE LIMIT must precede PAST LIMIT when both are used
  • Fixes example SQL to use the correct clause order
  • Adds missing retention_future_limit and retention_past_limit production rule definitions to the spec
  • Reorders FUTURE/PAST LIMIT section descriptions in manage-database.md to match syntax order
  • Fixes broken glossary anchor: #replication-factor#replication-factor-rf

Closes #6590
Stacked on #6945

…loses #6590)

The correct InfluxQL syntax has always been FUTURE LIMIT before PAST LIMIT.
The old docs incorrectly showed PAST before FUTURE in grammar definitions,
examples, and section ordering. Also fixes a broken glossary anchor link
for replication factor and adds missing production rule definitions to the
spec.
@jstirnaman jstirnaman requested a review from a team as a code owner March 14, 2026 20:13
@jstirnaman jstirnaman requested review from sanderson and removed request for a team March 14, 2026 20:13
@github-actions github-actions bot added the product:v1-enterprise InfluxDB Enterprise v1 label Mar 14, 2026
@github-actions
Copy link
Contributor

PR Preview Action v1.4.8
🚀 Deployed preview to https://influxdata.github.io/docs-v2/pr-preview/pr-6947/
on branch gh-pages at 2026-03-14 20:15 UTC

Copy link
Contributor Author

@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.

👍

@github-actions
Copy link
Contributor

Preview Pages for Review

2 page(s) changed in this PR:

Preview URLs

@github-copilot please review the preview pages listed above using the template below:

Visual Review Prompt

Review the rendered documentation pages at the preview URLs listed below.
Check each page for visual and structural issues that are invisible in the
Markdown source.

Checklist

For each preview URL, verify:

  • No raw shortcodes — No {{< or {{% syntax visible on the page
  • No placeholder text — No PLACEHOLDER, TODO, FIXME, or
    template variables visible in rendered content
  • Layout intact — No overlapping text, missing images, or collapsed
    sections
  • Code blocks render correctly — No raw HTML fences or Markdown
    syntax visible inside code blocks
  • Product names correct — Page header, breadcrumbs, and sidebar show
    the correct product name
  • No 404s or errors — Page loads without error states
  • Navigation correct — Sidebar entries link to the right pages and
    the page appears in the expected location

Output

Follow the shared review comment format, severity definitions, and label
mapping in
templates/review-comment.md.

Adapt the "Files Reviewed" section to list preview URLs instead of file
paths.

Preview URLs

@jstirnaman jstirnaman merged commit 48acce4 into worktree-jts-fix-unpublish-ent-1.12.3 Mar 14, 2026
9 of 10 checks passed
@jstirnaman jstirnaman deleted the jts-enterprise-v1.12.2-fixes branch March 14, 2026 20:36
github-actions bot added a commit that referenced this pull request 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant