fix: close mysql statement and rows resources when sql exec end (#1720)

* fix: close mysql statement and rows resources when sql exec end

* fix: close mysql statement and rows resources when sql exec end

* style: move code to other place

* style: correct the typo(Prepare)

Co-authored-by: 陈文军 <[email protected]>

Author: chenwenjun-github

pkg/db/v1beta1/mysql/mysql.go

@@ -129,9 +129,12 @@ func (d *dbConn) RegisterObservationLog(trialName string, observationLog *v1beta
 	// Prepare the statement
 	stmt, err := d.db.Prepare(sqlQuery)
 	if err != nil {
-		return fmt.Errorf("Pepare SQL statement failed: %v", err)
+		return fmt.Errorf("Prepare SQL statement failed: %v", err)
 	}
 
+	// Close the statement
+	defer stmt.Close()
+
 	// Execute INSERT
 	_, err = stmt.Exec(values...)
 	if err != nil {
@@ -176,6 +179,8 @@ func (d *dbConn) GetObservationLog(trialName string, metricName string, startTim
 	if err != nil {
 		return nil, fmt.Errorf("Failed to get ObservationLogs %v", err)
 	}
+	// Close the rows
+	defer rows.Close()
 	result := &v1beta1.ObservationLog{
 		MetricLogs: []*v1beta1.MetricLog{},
 	}