fix: Linter fixes for plugins/inputs/m*

[zak-pawel]

Following findings were fixed:

plugins/inputs/mailchimp/chimp_api.go:8:2           revive         imports-blacklist: should not use the following blacklisted import: "log"
plugins/inputs/marklogic/marklogic.go:166:66        revive         import-shadowing: The name 'url' shadows an import name
plugins/inputs/marklogic/marklogic.go:228:36        revive         import-shadowing: The name 'url' shadows an import name
plugins/inputs/mcrouter/mcrouter.go:149:1           revive         confusing-results: unnamed results of the same type may be confusing, consider using named results
plugins/inputs/mesos/mesos.go:25:2                  staticcheck    SA9004: only the first constant in this group has an explicit type
plugins/inputs/mesos/mesos.go:7:2                   revive         imports-blacklist: should not use the following blacklisted import: "log"
plugins/inputs/minecraft/client.go:48:2             revive         import-shadowing: The name 'rcon' shadows an import name
plugins/inputs/mongodb/mongostat.go:1314:13         revive         superfluous-else: if block ends with a break statement, so drop this else and outdent its block
plugins/inputs/mongodb/mongostat.go:906:1           revive         confusing-results: unnamed results of the same type may be confusing, consider using named results
plugins/inputs/monit/monit.go:17:2                  staticcheck    SA9004: only the first constant in this group has an explicit type
plugins/inputs/monit/monit.go:247:2                 revive         early-return: if c {...} else {... return } can be simplified to if !c { ... return } ...
plugins/inputs/mqtt_consumer/mqtt_consumer.go:75:2  revive         confusing-naming: Field 'topicTag' differs only by capitalization to other field in the struct type MQTTConsumer
plugins/inputs/mysql/mysql.go:1819:3                revive         defer: prefer not to defer inside loops
plugins/inputs/mysql/mysql.go:947:27                sqlclosecheck  Rows/Stmt was not closed

[srebhan]

Thanks @zak-pawel! Only two minor comments/suggestions. Feel free to ignore them if you don't like them. :-)

[telegraf-tiger]

🥳 This pull request decreases the Telegraf binary size by -0.01 % for linux amd64 (new size: 132.1 MB, nightly size 132.1 MB)

📦 Looks like new artifacts were built from this PR.

Expand this list to get them here! 🐯

Artifact URLs

DEB	RPM	TAR GZ	ZIP
amd64.deb	aarch64.rpm	darwin_amd64.tar.gz	windows_amd64.zip
arm64.deb	armel.rpm	freebsd_amd64.tar.gz	windows_i386.zip
armel.deb	armv6hl.rpm	freebsd_armv7.tar.gz
armhf.deb	i386.rpm	freebsd_i386.tar.gz
i386.deb	ppc64le.rpm	linux_amd64.tar.gz
mips.deb	s390x.rpm	linux_arm64.tar.gz
mipsel.deb	x86_64.rpm	linux_armel.tar.gz
ppc64el.deb		linux_armhf.tar.gz
s390x.deb		linux_i386.tar.gz
		linux_mips.tar.gz
		linux_mipsel.tar.gz
		linux_ppc64le.tar.gz
		linux_s390x.tar.gz
		static_linux_amd64.tar.gz

[srebhan]

LGTM. Thank you @zak-pawel for silencing another bunch of those...

[powersj]

Thanks for this! There are a couple of logic-like changes where you moved functions around, but I think it all still looks good. Thanks again!