mfocuz/zabbix_plugins — pull request #1 by Suic (2014-04-09 13:57:54+00:00)
Title: Removed finish call

https://metacpan.org/pod/DBI#finish
Adding calls to finish after loop that fetches all rows is a common mistake, don't do it, it can mask genuine problems like uncaught fetch errors.

When all the data has been fetched from a SELECT statement, the driver will automatically call finish for you. So you should not call it explicitly except when you know that you've not fetched all the data from a statement handle and the handle won't be destroyed soon.