Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SQL & InfluxQL reference docs for Pro and OSS v3 #5718

Open
wants to merge 3 commits into
base: monolith
Choose a base branch
from

Conversation

sanderson
Copy link
Collaborator

Uses shared content to add SQL and InfluxQL reference docs to InfluxDB Pro and InfluxDB OSS v3 docs.

  • Rebased/mergeable

@sanderson sanderson requested a review from jstirnaman December 19, 2024 17:30
@sanderson sanderson changed the title Monolith lang ref SQL & InfluxQL reference docs for Pro and OSS v3 Dec 19, 2024
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.

🔥 This looks great! Just a few minor fixes and thoughts about eliminating "Misc operator" pages in the future (not to hold this up).

title: InfluxQL transformation functions
list_title: Transformation functions
description: >
Use transformation functions modify and return values in each row of queried data.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Use transformation functions modify and return values in each row of queried data.
Use transformation functions to modify and return values in each row of queried data.

---
title: JOIN clause
description: >
Use the `JOIN` clause to join to data from different tables together.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Use the `JOIN` clause to join to data from different tables together.
Use the `JOIN` clause to join together data from different tables.

list_title: Comparison operators
description: >
Comparison operators evaluate the relationship between the left and right
operands and returns `true` or `false`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
operands and returns `true` or `false`.
operands and return `true` or `false`.

title: InfluxQL transformation functions
list_title: Transformation functions
description: >
Use transformation functions modify and return values in each row of queried data.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Use transformation functions modify and return values in each row of queried data.
Use transformation functions to modify and return values in each row of queried data.

---
title: JOIN clause
description: >
Use the `JOIN` clause to join to data from different tables together.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Use the `JOIN` clause to join to data from different tables together.
Use the `JOIN` clause to join together data from different tables.

list_title: Comparison operators
description: >
Comparison operators evaluate the relationship between the left and right
operands and returns `true` or `false`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
operands and returns `true` or `false`.
operands and return `true` or `false`.

list_code_example: |
| Operator | Meaning | Example | Result |
| :------------: | :----------------------- | :-------------------------------------- | :------------ |
| `\|\|` | Concatenate strings | `'Hello' \|\| ' world'` | `Hello world` |
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this be in a separate "Strings operators" page?

| Operator | Meaning | Example | Result |
| :------------: | :----------------------- | :-------------------------------------- | :------------ |
| `\|\|` | Concatenate strings | `'Hello' \|\| ' world'` | `Hello world` |
| `AT TIME ZONE` | Apply a time zone offset | _[View example](/influxdb/v3/reference/sql/operators/other/#at-time-zone)_ | |
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this be in a separate "Time operators" page?

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.

2 participants