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

fix: OSS API documentation documents InfluxDB Cloud parameterized queries feature #646

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jstirnaman
Copy link
Contributor

@jstirnaman jstirnaman commented Feb 15, 2024

Fixes influxdata/docs-v2#5320 for the OpenAPI contracts. Docs fix pending approval here.

  • Effectively removes property from OSS Query API.
  • Re-diverges Query schema, removing from OSS and keeping it in Cloud.
  • Re-diverges /query paths, using specific schemas in oss and cloud paths.

@jstirnaman jstirnaman requested a review from a team as a code owner February 15, 2024 22:11
@jstirnaman jstirnaman requested review from sanderson and removed request for a team February 15, 2024 22:11
@jstirnaman jstirnaman marked this pull request as draft February 15, 2024 22:11
Copy link
Contributor

@sanderson sanderson left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

…ata/docs-v2#5320)

- Effectively removes  property from OSS Query API.
- Re-diverges Query schema, removing  from OSS and keeping it in Cloud.
- Re-diverges paths with oss- and cloud-specific schemas.
@jstirnaman jstirnaman marked this pull request as ready for review February 15, 2024 23:03
jstirnaman added a commit to influxdata/docs-v2 that referenced this pull request Feb 15, 2024
…a. OSS doesn't support parameterized queries. See upstream PR: influxdata/openapi#646
jstirnaman added a commit to influxdata/docs-v2 that referenced this pull request Feb 15, 2024
…a. OSS doesn't support parameterized queries. See upstream PR: influxdata/openapi#646 (#5325)
@jstirnaman jstirnaman force-pushed the fix/oss-query-schema branch 3 times, most recently from c430e14 to 2a0144a Compare May 16, 2024 17:36
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.

OSS API documentation documents InfluxDB Cloud features
2 participants