CREATE RETENTION and ALTER RETENTION for V1 have errors around LIMIT clauses

[davidby-influx]

CREATE RETENTION POLICY statements with PAST LIMIT and FUTURE LIMIT clauses must have the FUTURE LIMIT clause, if it exists, before the PAST LIMIT clause.

The LIMIT clauses are undocumented for ALTER RETENTION

Relevant URLs

• Modify retention policies with ALTER RETENTION POLICY
• Create retention policies with CREATE RETENTION POLICY
• Tested examples

[jstirnaman]

jts-fix-rp-limit-clauses

Overview

Fix documentation errors for CREATE RETENTION POLICY and ALTER RETENTION POLICY InfluxQL statements:

1. The FUTURE LIMIT clause must come before PAST LIMIT (parser requirement)
2. The FUTURE LIMIT and PAST LIMIT clauses were missing from ALTER RETENTION POLICY documentation

Progress

• Phase 1: Research - Verified correct syntax order from InfluxQL parser (FUTURE LIMIT before PAST LIMIT)
• Phase 2: Implementation - Fixed 4 files across OSS v1 and Enterprise v1
• Phase 3: Testing - Hugo build passes, all syntax corrected

Changes Made

Files Modified

1. content/influxdb/v1/query_language/manage-database.md
2. content/enterprise_influxdb/v1/query_language/manage-database.md
3. content/influxdb/v1/query_language/spec.md
4. content/enterprise_influxdb/v1/query_language/spec.md

Specific Fixes

• Corrected clause order: FUTURE LIMIT before PAST LIMIT in all syntax examples
• Added FUTURE LIMIT and PAST LIMIT clauses to ALTER RETENTION POLICY documentation
• Added grammar definitions for retention_future_limit and retention_past_limit
• Updated retention_policy_option to include limit clauses
• Added ALTER RETENTION POLICY example with LIMIT clauses

Notes

Cross-branch context and research notes go here.

Per parser_test.go:

CREATE RETENTION POLICY policy1 ON testdb DURATION 1h REPLICATION 2 SHARD DURATION 1s FUTURE LIMIT 12h PAST LIMIT 3s

[davidby-influx]

See my notes in the PR: fix(influxdb-v1): correct FUTURE LIMIT and PAST LIMIT clause order and add to ALTER RETENTION POLICY #6591

• FUTURE LIMIT must come before PAST LIMIT in both ALTER and CREATE` statements
• CREATE RETENTION POLICY with limits works correctly in 1.12.2 for both OSS and Enterprise
• ALTER RETENTION POLICY works correctly in OSS 1.12.2, but does not work in 1.12.2 Enterprise
• ALTER RETENTION POLICY will work correctly for all versions of 1.12.3

We will need the Docs to differ for OSS in 1.12.2 (by containing ALTER with LIMIT clauses) but be re-unified in 1.12.3 when the bug is fixed in Enterprise.

[jstirnaman]

See https://github.com/influxdata/plutonium/pull/4387