Skip to content

[do not merge] SQL GA#570

Draft
kbatuigas wants to merge 1 commit intomainfrom
rp-sql
Draft

[do not merge] SQL GA#570
kbatuigas wants to merge 1 commit intomainfrom
rp-sql

Conversation

@kbatuigas
Copy link
Copy Markdown
Contributor

@kbatuigas kbatuigas commented May 1, 2026

  • Add migrated Oxla docs

  • New SQL docs for GA (will be continuously merged into rp-sql branch leading up to GA release)

Page previews
https://deploy-preview-570--rp-cloud.netlify.app/redpanda-cloud/sql/
https://deploy-preview-570--rp-cloud.netlify.app/redpanda-cloud/reference/sql/

* Add converted SQL reference files

* Duplicate file

* Add index files

* Add SQL reference to Cloud nav

* Move reference pages to reference module

* Convert remaining docs

* Add placeholder index files

* Cleanup

* Move SQL section after RPCN

* Continue cleanup using plugin audit and review

* External links open in new window

* Minor edit to trigger deploy preview

* Migrate degraded state doc

* Minor style edits

* Address drift from Oxla changes since mid-Oct 2025

* Include CREATE TABLE and DROP changes

* Update existing pages with placeholders for pages to be created

* Another pass at style fixes

* Missing TODO

* Add remaining new reference pages to address doc drift

* Remove Oxla storage compression references

* Remove INSERT/UPDATE/DELETE from psycopg2 supported features

* Remove Oxla-specific reasons from java-jdbc unsupported list

* Remove DELETE/UPDATE references from php-pdo rowCount

* Remove CREATE TABLE/INSERT example from select.adoc

Assume that a student_data table is already created and populated with data

* Remove table duplication TIP

* Fix clumped "SELECTstatement"

* Add missing Venn diagrams to JOIN pages

* Inconsistent formatting with function names and parentheses

* Incorrect COUNT logic with HAVING

* Add NULLS FIRST/LAST, fix interval sorting

* Replace CREATE TABLE oxlafunctions example in offset.adoc

Redpanda SQL does not support CREATE TABLE without an external catalog

Example assumes precreated salaryemp table

* Unrelated "employees" text

* Move column compatibility note to set-operations index

Column type compatibility and flexible ordering
applies to all set operations (UNION, INTERSECT, EXCEPT)

* Both ROWS and RANGE frame modes are supported

* Add GEOMETRY/GEOGRAPHY

* Create sql-operators section with full operator list

* timestamp-with-time-zone formatting

* Remove internal memory representation for date type

* Add month and day conversion note

* Remove internal memory representation from bool type

* System catalogs disambiguation note and 7 new pg_* pages

* Fix nav

* Move bitwise operators

* Double check SQL examples and outputs

* Remove unix_macros

* Reorganize math and trig function pages

* Implement some deferred fixes related to old Oxla examples

* Fix anchors in xrefs

* Apply QA checklist first pass across SQL reference docs

Run 11-point QA checklist against all migrated SQL pages (148 files).
Fixes applied: remove "Please note" phrasing, fix "here" link text,
replace future tense output descriptions with present tense, simplify
"allows/enables you to" constructions, remove minimizing language,
fix wordy instructions, add missing admonition punctuation, replace
verbose output lead-ins, remove "etc.", normalize positional references,
and update stale PostgreSQL doc links to /docs/current/.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Person 4 edits, including future tense, passive voice

* more Person 4 edits

* Person 4 edits

* Parallelize :description: metadata for SQL language client pages

Standardize the Java, PHP, and Python connector pages on the same
imperative "Connect to Redpanda SQL from <language> using <tool>"
template already used by the C# page and the parent index page.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Apply Feediver1 review suggestions

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Apply pending PR suggestion comments

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Minor edit - statements index

* keywords.adoc - move in nav tree, make table more scannable

* alter-table minor edit

* create-redpanda-catalog: Improve Options section

* create-storage minor edit

* Improve CREATE TABLE reference page clarity

Clarify NOTE about CREATE TABLE behavior, convert multi-value option
descriptions to lists for scannability, and rename ambiguous example
heading.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* select: review fixes

* set-show: review fixes

* show-tables minor edit

* show-nodes: review fixes

* describe: review fixes

* sql-statements review pass

* from - review pass

* Minor edit

* set-operations: review fixes

* More review fixes for sql-clauses

* other-functions review fixes

* Normalize QA fixes

* Apply edits based on doc team review

* Standardize "To do X, run:"

* Structural improvements

* Sweep for passive voice

* Person 2 style review: aggregate, window, JSON functions

Style fixes across 31 files:
- Future tense → present tense ("will return" → "returns")
- Wordy instructions ("lets you" → direct verb)
- Awkward phrasing ("Here's how" → cleaner alternatives)
- First person ("we're going to" → "This example")
- Passive "It" patterns in tables → active voice
- Positional references ("the above query" → "Output:")
- Typo fix ("ouput" → "output")

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Pass to standardize QA

* Apply suggestions from review

* Minor edits

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: micheleRP <michele@redpanda.com>
Co-authored-by: Joyce Fee <joyce@redpanda.com>
Co-authored-by: JakeSCahill <jake@redpanda.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 1, 2026

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit 122a0e0
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/69f4ffd9c94da90008e3f684
😎 Deploy Preview https://deploy-preview-570--rp-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 1, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3f94229-096c-4ac1-8760-4162f269cba0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rp-sql

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant