Minor fixes for go1.10 vet

go vet is a little stricter in go1.10. This fixes two new classes
of issues that it picks up:
- "Println arg list ends with redundant newline"
  (dicussed in golang/go#18085)
- "Errorf format %d arg res.RowsAffected is a func value, not called"

Author: nvanbenschoten

src/go/src/rubrik/sqlapp/filesystem_simulator/log.go

@@ -213,7 +213,7 @@ func (r *replayedLog) printRecursive(uuid string) {
 	sort.Strings(childNames)
 	// TODO change in case of tree structure directory will need path
 	fmt.Println(uuid)
-	fmt.Println(childNames, "\n")
+	fmt.Print(childNames, "\n\n")
 	for _, childUUID := range childUUIDs {
 		if fileTypeDirectory == (r.uuidToFile[childUUID]).typ {
 			r.printRecursive(childUUID)

src/go/src/rubrik/sqlapp/filesystem_simulator/main.go

@@ -194,7 +194,7 @@ func execOneRow(
 	if n, err = res.RowsAffected(); err != nil {
 		return err
 	} else if n != 1 {
-		return fmt.Errorf("%s affected %d rows, expected 1", msgPrefix, res.RowsAffected)
+		return fmt.Errorf("%s affected %d rows, expected 1", msgPrefix, n)
 	}
 	return nil
 }
@@ -457,7 +457,7 @@ func printDirectoriesRecursive(
 	}
 	sort.Strings(childNames)
 	fmt.Println(path)
-	fmt.Println(childNames, "\n")
+	fmt.Print(childNames, "\n\n")
 
 	for _, childName := range childNames {
 		childUUID := childMap[childName]