fix: Show sync time (#2493)

disq · web-flow · commit 014e8aadcc43 · 2022-10-08T15:34:19.000Z
old look:
&gt; Summary: resources: 22029, errors: 100, panic: 0 failed_writes: 0

new look:
&gt; Summary: resources: 25660, errors: 100, panic: 0, failed_writes: 0, time: 2m42s
diff --git a/cli/cmd/sync.go b/cli/cmd/sync.go
@@ -185,9 +185,11 @@ func syncConnection(ctx context.Context, sourceSpec specs.Source, destinationsSp
 		return fmt.Errorf("failed to get sync summary: %w", err)
 	}
 	_ = bar.Finish()
+	tt := time.Since(syncTime)
+
 	fmt.Println("Sync completed successfully.")
-	fmt.Printf("Summary: resources: %d, errors: %d, panic: %d failed_writes: %d\n", totalResources, summary.Errors, summary.Panics, failedWrites)
+	fmt.Printf("Summary: resources: %d, errors: %d, panic: %d, failed_writes: %d, time: %s\n", totalResources, summary.Errors, summary.Panics, failedWrites, tt.Truncate(time.Second).String())
 	log.Info().Str("source", sourceSpec.Name).Strs("destinations", sourceSpec.Destinations).
-		Int("resources", totalResources).Uint64("errors", summary.Errors).Uint64("panic", summary.Panics).Uint64("failedWrites", failedWrites).Msg("sync completed successfully")
+		Int("resources", totalResources).Uint64("errors", summary.Errors).Uint64("panic", summary.Panics).Uint64("failed_writes", failedWrites).Float64("time_took", tt.Seconds()).Msg("Sync completed successfully")
 	return nil
 }

Original file line number	Diff line number	Diff line change
`@@ -185,9 +185,11 @@ func syncConnection(ctx context.Context, sourceSpec specs.Source, destinationsSp`
`185`	`185`	`return fmt.Errorf("failed to get sync summary: %w", err)`
`186`	`186`	`}`
`187`	`187`	`_ = bar.Finish()`
	`188`	`+ tt := time.Since(syncTime)`
	`189`	`+`
`188`	`190`	`fmt.Println("Sync completed successfully.")`
`189`		`- fmt.Printf("Summary: resources: %d, errors: %d, panic: %d failed_writes: %d\n", totalResources, summary.Errors, summary.Panics, failedWrites)`
	`191`	`+ fmt.Printf("Summary: resources: %d, errors: %d, panic: %d, failed_writes: %d, time: %s\n", totalResources, summary.Errors, summary.Panics, failedWrites, tt.Truncate(time.Second).String())`
`190`	`192`	`log.Info().Str("source", sourceSpec.Name).Strs("destinations", sourceSpec.Destinations).`
`191`		`- Int("resources", totalResources).Uint64("errors", summary.Errors).Uint64("panic", summary.Panics).Uint64("failedWrites", failedWrites).Msg("sync completed successfully")`
	`193`	`+ Int("resources", totalResources).Uint64("errors", summary.Errors).Uint64("panic", summary.Panics).Uint64("failed_writes", failedWrites).Float64("time_took", tt.Seconds()).Msg("Sync completed successfully")`
`192`	`194`	`return nil`
`193`	`195`	`}`