-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add section explaining table metadata #774
Conversation
Codecov Report
@@ Coverage Diff @@
## main #774 +/- ##
==========================================
- Coverage 93.39% 93.20% -0.20%
==========================================
Files 43 43
Lines 1846 1853 +7
Branches 230 232 +2
==========================================
+ Hits 1724 1727 +3
- Misses 96 98 +2
- Partials 26 28 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@utkarsharma2 it is probably worth explaining where we retrieve the metadata information from when the user doesn't set this property!
Co-authored-by: Tatiana Al-Chueyr <tatiana.alchueyr@gmail.com>
Co-authored-by: Tatiana Al-Chueyr <tatiana.alchueyr@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggest changing the tag to metadata_example_snowflake
than metadata_example_1
for better readability
python-sdk/example_dags/example_amazon_s3_snowflake_transform.py
Outdated
Show resolved
Hide resolved
python-sdk/example_dags/example_amazon_s3_snowflake_transform.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com>
Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com>
Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com>
Co-authored-by: Phani Kumar <94376113+phanikumv@users.noreply.github.com>
Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
Description
What is the current behavior?
Some of the examples are in https://astro-sdk-python.readthedocs.io/en/stable/astro/sql/operators/load_file.htm. use the metadata parameter when creating a Table, but AFAICT we don't explain what that does.
closes: #767
What is the new behavior?
Added section in docs explaining the usage of Metadata.
Does this introduce a breaking change?
No
Checklist