No print statements in exec modules please
Captalize class names please
Collapse empty pass statements
This appears to be a no-op
Remove pylint resulting in no-op overhead
Helper function is no longer used at all
Import logging
Remove extra check for None because we can compare str to None.

@thatch45 please review. Perhaps this was for performance?
Collapse no-op matching
Change mysql pillar test to use updated class name
Bad logic error on my part
Merge pull request #19246 from cachedout/more_cleanup

More cleanup