Fix Pgbench: serialization/deadlock failure record in the stats

Marina Polyakova · Marina Polyakova · commit ab07a0450c32 · 2017-08-21T16:22:06.000+03:00
A failure should be reported only once, and not after a failure and after the
end of a failed transaction block.
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
@@ -2150,7 +2150,8 @@ doCustom(TState *thread, CState *st, StatsData *agg)
 					}
 				}
 
-				if (st->last_begin_state->attempts_number ==
+				if ((serialization_failure || deadlock_failure) &&
+					st->last_begin_state->attempts_number ==
 					max_attempts_number)
 				{
 					/*

Original file line number	Diff line number	Diff line change
`@@ -2150,7 +2150,8 @@ doCustom(TState thread, CState st, StatsData *agg)`
`2150`	`2150`	`}`
`2151`	`2151`	`}`
`2152`	`2152`
`2153`		`- if (st->last_begin_state->attempts_number ==`
	`2153`	`+ if ((serialization_failure \|\| deadlock_failure) &&`
	`2154`	`+ st->last_begin_state->attempts_number ==`
`2154`	`2155`	`max_attempts_number)`
`2155`	`2156`	`{`
`2156`	`2157`	`/*`