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
[CONTRIB] fixing null eq safe for SQLAlchemy and rendering #8617
[CONTRIB] fixing null eq safe for SQLAlchemy and rendering #8617
Conversation
✅ Deploy Preview for niobium-lead-7998 canceled.
|
Hi @kenwade4 the checks are failing at cloud-tests with error "E great_expectations.exceptions.exceptions.DataContextError: "GX_CLOUD_ORGANIZATION_ID", "GX_CLOUD_ACCESS_TOKEN" arg(s) required for ge_cloud_mode but neither provided nor found in environment or in global configs ("/home/runner/.great_expectations/great_expectations.conf", "/etc/great_expectations.conf")." |
Hi @karthigaiselvanm sorry about the delay on this one. I'm picking up the task from Ken.
|
column_list[idx] != column_list[0], | ||
sa.and_(column_list[idx].is_(None), column_list[0].isnot(None)), | ||
sa.and_(column_list[idx].isnot(None), column_list[0].is_(None)), |
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.
Hi @karthigaiselvanm thank you very much for this. Rendering and the null
case both look great
b0290e2
into
great-expectations:develop
invoke lint
(usesblack
+ruff
)For more information about contributing, see Contribute.
After you submit your PR, keep the page open and monitor the statuses of the various checks made by our continuous integration process at the bottom of the page. Please fix any issues that come up and reach out on Slack if you need help. Thanks for contributing!