Closed lecardozo closed 2 years ago
Hey, @gucciwang!
Wanted to hear your thoughts on this PR. Let me know if there's anything else we could do to enable this support for Deequ>=2.x 😃
Thanks!
Can't wait for the fix of #70
🥺
Is there a test report (if not from CI) that we can review? Just to make sure everything passes.
@chenliu0831 You can check the reports of tests here while I'm addressing your suggestions. As soon as we merge it, I'd expect to see them running on awslabs/pydeequ. :)
@chenliu0831 You can check the reports of tests here while I'm addressing your suggestions. As soon as we merge it, I'd expect to see them running on awslabs/pydeequ. :)
Thanks, please just post a final link again when you are ready, I will review & approve
Done! Here is the last run of the CI workflow for c410f72
Hi are you going to release the python package soon ?
Hi, I would love to use this library. Does this PR support for spark 3.2.1, databricks runtime 10.4 LTS?
Related issues
31
70
90
Related PRs
96
75
32
Changes
In order to support latest deequ releases, a few adjustments were necessary:
CategoricalRangeRule
andFractionalRangeRule
to include extra arguments for deequ>=2.0.0.Additionally, a few technical debts were addressed
SPARK_VERSION
environment variable)By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.