[CT-271] [Feature] not_null test selects column instead of *

[willbowditch]

resolves #4769

Description

Currently the not_null test writes SQL equivalent to

SELECT *
FROM model
WHERE column IS NULL

Which on columnar databases like BigQuery makes running the test more expensive than

SELECT column
FROM model
WHERE column IS NULL

This PR changes the test to SELECT column, unless the store failures feature is enabled, in which case it is useful to keep the failing rows in totality using SELECT *.

Checklist

• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have updated the CHANGELOG.md and added information about my change

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.

CLA has not been signed by users: @willbowditch

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.

CLA has not been signed by users: @willbowditch

[willbowditch]

@drewbanin I have signed the CLA.

No changes to tests in the PR at the moment, but happy to have a go at adding some if they're needed.

[leahwicz]

@cla-bot check

[cla-bot]

The cla-bot has been summoned, and re-checked this pull request!

[ChenyuLInx]

Looks good to me. Thanks for contributing this!!

[leahwicz]

@willbowditch I had to close and reopen the PR to pick up on some new GitHub Checks that we just added. One is around how we do the Changelog and that's what failing here. So sorry for the inconvenience! Would you mind updating the Changelog entry to follow our new process that you can see documented here

[willbowditch]

Hey @leahwicz - no worries, I've reverted the changes to the markdown changelog and pushed up the file changie created