Re: Stop the search once replication origin is found

2023-11-22 Thread Peter Smith
On Wed, Nov 22, 2023 at 7:49 PM Amit Kapila wrote: > > On Mon, Nov 20, 2023 at 4:36 PM Amit Kapila wrote: > > > > On Mon, Nov 20, 2023 at 2:36 PM Antonin Houska wrote: > > > > > > Although it's not performance-critical, I think it just makes sense to > > > break > > > the loop in replorigin_ses

Re: Stop the search once replication origin is found

2023-11-22 Thread Amit Kapila
On Mon, Nov 20, 2023 at 4:36 PM Amit Kapila wrote: > > On Mon, Nov 20, 2023 at 2:36 PM Antonin Houska wrote: > > > > Although it's not performance-critical, I think it just makes sense to break > > the loop in replorigin_session_setup() as soon as we've found the origin. > > > > Your proposal sou

Re: Stop the search once replication origin is found

2023-11-20 Thread Amit Kapila
On Mon, Nov 20, 2023 at 2:36 PM Antonin Houska wrote: > > Although it's not performance-critical, I think it just makes sense to break > the loop in replorigin_session_setup() as soon as we've found the origin. > Your proposal sounds reasonable to me. -- With Regards, Amit Kapila.

Stop the search once replication origin is found

2023-11-20 Thread Antonin Houska
Although it's not performance-critical, I think it just makes sense to break the loop in replorigin_session_setup() as soon as we've found the origin. -- Antonin Houska Web: https://www.cybertec-postgresql.com diff --git a/src/backend/replication/logical/origin.c b/src/backend/replication/logica