-
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
[MAINTENANCE] Run spark and onboarding data assistant test in their own jobs. #5951
Conversation
billdirks
commented
Sep 7, 2022
•
edited
Loading
edited
- My code follows the Great Expectations style guide
- I have performed a self-review of my own code
- I have commented my code, particularly in hard-to-understand areas
- I have made corresponding changes to the documentation
- I have added unit tests where applicable and made sure that new and existing tests are passing.
- I have run any local integration tests and made sure that nothing is broken.
✅ Deploy Preview for niobium-lead-7998 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
👇 Click on the image for a new way to code review
Legend |
b4cdb3e
to
7154098
Compare
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.
LGTM! One tiny nit
…batch-notebook' of https://github.com/great-expectations/great_expectations into m/GREAT-465/GREAT-1226/GREAT-1227/configured-sql-multi-batch-notebook * 'm/GREAT-465/GREAT-1226/GREAT-1227/configured-sql-multi-batch-notebook' of https://github.com/great-expectations/great_expectations: [MAINTENANCE] More unit tests for `Stores` (#5953) [MAINTENANCE] Unit tests for `ValidationGraph` and related classes (#5954) [MAINTENANCE] Run spark and onboarding data assistant test in their own jobs. (#5951)
…r-usage-stats-opt * develop: (62 commits) [MAINTENANCE] Bump `Marshmallow` upper bound to work with Airflow operator (#5952) [MAINTENANCE] Add x-fails to flaky Cloud tests for purposes of 0.15.22 (#5964) [BUGFIX] Prevent "division by zero" errors in Rule-Based Profiler calculations when Batch has zero rows (#5960) [FEATURE] Improve slack error condition (#5818) [MAINTENANCE] More unit tests for `Stores` (#5953) [MAINTENANCE] Unit tests for `ValidationGraph` and related classes (#5954) [MAINTENANCE] Run spark and onboarding data assistant test in their own jobs. (#5951) [BUGFIX] prefix and suffix asset names are only relevant for InferredSqlAlchemyDataConnector (#5950) Add a dev-tools requirements option (#5944) [BUGFIX] Data Assistant plotting with zero expectations produced (#5934) [MAINTENANCE] Unit tests for `ConfigurationStore` (#5948) [BUGFIX] Don't include abstract Expectation classes in _retrieve_expectations_from_module (#5947) [BUGFIX] Making an all-NULL column handling in RuleBasedProfiler more robust (#5937) [MAINTENANCE] Run comprehensive tests in a random order (#5942) [MAINTENANCE] Update to OnboardingDataAssistant Notebook - Sql (#5939) [MAINTENANCE] Add missing import for ConfigurationIdentifier (#5943) [MAINTENANCE] Mark tests within `tests/rule_based_profiler` (#5930) [MAINTENANCE] Move `Store` test utils from source code to tests (#5932) [FEATURE] DataAssistants Example Notebook - Spark (#5919) [MAINTENANCE] Reset globals modified in tests (#5936) ...