Skip to content

Conversation

Alexandretubiana-marble
Copy link
Contributor

@Alexandretubiana-marble Alexandretubiana-marble commented Oct 3, 2025

  • Created new Processes folder
  • Created md file to describe process for data ingestion via bucket storage

Add md file to describe process for data ingestion via bucket storage
@Alexandretubiana-marble Alexandretubiana-marble marked this pull request as ready for review October 3, 2025 13:44
Copy link
Contributor

@Pascal-Delange Pascal-Delange left a comment

Choose a reason for hiding this comment

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

👍
One point: it's not necessary, and to be honest rather cumbersome for any future rewriting, to format markdown files with line breaks after 80-100 chars. Just let lines fold naturally, github display of markdown files works well enough without trying to do smart formatting.

Comment on lines +26 to +27
Once your CSV file is ready, you need to upload it to the designated cloud
storage bucket.
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
Once your CSV file is ready, you need to upload it to the designated cloud
storage bucket.
Once your CSV file is ready, you need to upload it to the object storage bucket you have configured as `INGESTION_BUCKET_URL`

Comment on lines +66 to +67
- **Confirm completion:** Allow several minutes for the ingestion to finish. The
time required will depend on the size of your file.
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
- **Confirm completion:** Allow several minutes for the ingestion to finish. The
time required will depend on the size of your file.
- **Confirm completion:** Wait for the ingestion to finish. The time required will depend on the size of your file, and should be between 1 and 10ms per row. Note that the ingestion job will by default time out after one hour.

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