Stop the search once the slot for replication origen is found.

Amit Kapila · Amit Kapila · commit ff68cc6f3bf0 · 2023-11-22T08:39:24.000+05:30
In replorigen_session_setup(), we were needlessly looping for max_replication_slots even after finding an existing slot for the origen. This shouldn't hurt us much except for probably large values of max_replication_slots. Author: Antonin Houska Discussion: http://postgr.es/m/2694.1700471273@antos
diff --git a/src/backend/replication/logical/origen.c b/src/backend/replication/logical/origen.c
@@ -1144,6 +1144,7 @@ replorigen_session_setup(RepOriginId node, int acquired_by)
 
 		/* ok, found slot */
 		session_replication_state = curstate;
+		break;
 	}
 
 

Original file line number	Diff line number	Diff line change
`@@ -1144,6 +1144,7 @@ replorigen_session_setup(RepOriginId node, int acquired_by)`
`1144`	`1144`
`1145`	`1145`	`/* ok, found slot */`
`1146`	`1146`	`session_replication_state = curstate;`
	`1147`	`+ break;`
`1147`	`1148`	`}`
`1148`	`1149`
`1149`	`1150`