removing todo TODO: Replace notnull constraint with a non-nullable en…

[pragmatical]

…tity constraint., as not sure what the driver is or what it is attempting to accomplish

Type of PR

• Documentation changes
• Code changes
• Test changes
• CI-CD changes
• GitHub Template changes

PR Checklist

• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My code follows the code style of this project.
• I ran lint checks locally prior to submission.
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Purpose of PR

Removed TODO, request does not make sense in the context and current implementation appears to accomplish what is desired

Does this introduce a breaking change

• YES
• NO

Validation

• Unit tests updated and ran successfully
• Update documentation or issue referenced above

Issues Closed or Referenced

• Closes #1066 (this will automatically close the issue when the PR closes)

[gortegaMS]

LGTM