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

Listing write steps for Core and Enterprise #5881

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

MeelahMe
Copy link
Contributor

@MeelahMe MeelahMe commented Mar 7, 2025

Closes #5878

Describe your proposed changes here.

Adding write steps for Core and Enterprise and describing the impact of flags and options on latency.

@MeelahMe MeelahMe added the InfluxDB v3 InfluxDB v3.0 (platforms using IOx) label Mar 7, 2025
@MeelahMe MeelahMe self-assigned this Mar 7, 2025
@MeelahMe MeelahMe marked this pull request as draft March 7, 2025 02:58
@MeelahMe
Copy link
Contributor Author

MeelahMe commented Mar 7, 2025

Are these additions close to what you were looking for? @jstirnaman

@jstirnaman
Copy link
Contributor

Are these additions close to what you were looking for? @jstirnaman

I like this and the detail explaining tradeoffs. IMO, the phrase "Key option" might be confusing. I would remove or replace it with a descriptive sentence.
I just merged a similar, but shorter variation of this - I wanted to add it to help explain no_sync in the API reference so I went ahead and copied it into /get-started. Sorry to step on your toes.

That said, /get-started is fairly bloated. I suggest we move this into a new page:

  1. Merge your version and the Data flow section from content/shared/v3-core-get-started into a content/shared/influxdb3-internals-reference/durability page, following the pattern in https://docs.influxdata.com/influxdb3/cloud-serverless/reference/internals/durability/.
  2. Remove all or most of the Data flow section from content/shared/v3-core-get-started and content/shared/v3-enterprise-get-started.
  3. Link to the new page from /get-started and /write-data.

@MeelahMe MeelahMe marked this pull request as ready for review March 10, 2025 22:40
@MeelahMe MeelahMe requested a review from jstirnaman March 10, 2025 22:41
@MeelahMe MeelahMe changed the title WIP: Listing write steps for Core and Enterprise Listing write steps for Core and Enterprise Mar 10, 2025
@@ -0,0 +1,17 @@
---
title: InfluxDB 3 Data durability
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

title: InfluxDB 3 Data durability
list_title: Data durability
description: >
The `influxdb3` CLI runs and interacts with the InfluxDB 3 Core server.
Copy link
Contributor

Choose a reason for hiding this comment

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

This description doesn't fit the page--seems like it's for the influxdb3 CLI reference.

@@ -0,0 +1,16 @@
---
title: InfluxDB 3 Internals
Copy link
Contributor

Choose a reason for hiding this comment

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

The file path should be content/influxdb3/core/reference/internals/_index.md

@@ -0,0 +1,17 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

The file path should be
content/influxdb3/core/reference/internals/durability/_index.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
InfluxDB v3 InfluxDB v3.0 (platforms using IOx)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

List write steps for Core and Enterprise and describe the impact of flags and options on latency.
2 participants