-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[DOCS] FDS Deployment Pattern - AWS S3 Pandas #7718
Conversation
✅ Deploy Preview for niobium-lead-7998 canceled.
|
<PythonTabTest /> | ||
|
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.
These changes are the result of the linter
…/great-expectations/great_expectations into m/_/deployment-pattern-s3-pandas * 'm/_/deployment-pattern-s3-pandas' of https://github.com/great-expectations/great_expectations: [MAINTENANCE] Update a Fluent Datasource related fixture name to better reflect its capabilities (#7725)
@@ -1863,7 +1863,7 @@ def main(): | |||
# any methods or classes you are adding to documentation with the @public_api | |||
# decorator and any relevant "new" or "deprecated" public api decorators. | |||
# If the actual is lower than the threshold, please reduce the threshold. | |||
PUBLIC_API_MISSING_THRESHOLD = 92 # TODO: reduce this number again once this works for the Fluent DS dynamic methods | |||
PUBLIC_API_MISSING_THRESHOLD = 99 # TODO: reduce this number again once this works for the Fluent DS dynamic methods |
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.
This had to be raised because I refer to more FDS methods in my script
tests/integration/docusaurus/deployment_patterns/aws_cloud_storage_pandas.py
Outdated
Show resolved
Hide resolved
tests/integration/docusaurus/deployment_patterns/aws_cloud_storage_pandas.py
Outdated
Show resolved
Hide resolved
…rage_pandas.py Co-authored-by: Gabriel <gabriel59kg@gmail.com>
…/great-expectations/great_expectations into m/_/deployment-pattern-s3-pandas * 'm/_/deployment-pattern-s3-pandas' of https://github.com/great-expectations/great_expectations: Update tests/integration/test_definitions/s3/integration_tests.py
This reverts commit 002add6.
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.
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.
This was as miss on my part. The section was referring to a different (spark s3 data context
) doc. Doc has been updated.
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.
Found a small issue with some seemingly unnecessary imports but other than that it looks good to me.
It might be worth adding a note or something that users can for the most part complete Part 2 "connecting to data" without completing all of Part 1.
…/great-expectations/great_expectations into m/_/deployment-pattern-s3-pandas * 'm/_/deployment-pattern-s3-pandas' of https://github.com/great-expectations/great_expectations: [MAINTENANCE] fix get available data assets names for fds (#7723)
Changes proposed in this pull request:
tests/integration/docusaurus/deployment_patterns/aws_cloud_storage_pandas.py
that tests the workflow.test_script_runner.py
Definition of Done