Adds support for verbose timing information. Closes #4237. Closes #4244

- JSON output format has changed to move the rows to under a `rows` property, with timing information under the `metadata` property
- Update timing display to show rows returned and rows fetched, as well as adding verbose mode which lists all scans
- Use enums for output mode and timing mode - timing is now either `on`, `off` or `verbose`
- Bugfix: ensure error is returned from ExecuteSystemClientCall. Closes #4246

pkg/interactive/interactive_client.go

@@ -397,7 +397,7 @@ func (c *InteractiveClient) executeQuery(ctx context.Context, queryCtx context.C
 	if err != nil {
 		error_helpers.ShowError(ctx, error_helpers.HandleCancelError(err))
 		// if timing flag is enabled, show the time taken for the query to fail
-		if cmdconfig.Viper().GetBool(constants.ArgTiming) {
+		if cmdconfig.Viper().GetString(constants.ArgTiming) != constants.ArgOff {
 			display.DisplayErrorTiming(t)
 		}
 	} else {