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

Improve docs on objectstorage #35294

Merged
merged 3 commits into from
Oct 31, 2023
Merged

Improve docs on objectstorage #35294

merged 3 commits into from
Oct 31, 2023

Conversation

bolkedebruin
Copy link
Contributor

Add more explanations, limitations and add example of attaching a filesystem.

cc @Taragolis


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Add more explanations, limitations and add example
of attaching a filesystem.
@@ -38,6 +44,21 @@ scheme.
it depends on ``aiobotocore``, which is not installed by default as it can create dependency
challenges with ``botocore``.

Cloud Object Stores are not real file systems
Copy link
Member

Choose a reason for hiding this comment

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

Big +10 on that. People do not understand that and have weird expectations about object storages.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

❤️ all the clarifications here. Very good update.

@potiuk potiuk merged commit 69bac3f into apache:main Oct 31, 2023
44 checks passed
@bolkedebruin bolkedebruin deleted the os_docs branch November 1, 2023 14:40
romsharon98 pushed a commit to romsharon98/airflow that referenced this pull request Nov 10, 2023
* Improve docs on objectstorage

Add more explanations, limitations and add example
of attaching a filesystem.
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Nov 19, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.8.0 milestone Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-58 area:providers changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) kind:documentation provider:amazon-aws AWS/Amazon - related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants