On Tue, Mar 16, 2021 at 6:22 PM vignesh C <vignes...@gmail.com> wrote: > > On Mon, Mar 15, 2021 at 6:14 PM Ajin Cherian <itsa...@gmail.com> wrote: > > > > On Mon, Mar 15, 2021 at 2:04 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > >> > > 2) table_states_not_ready global variable is used immediately after > call to FetchTableStates, we can make FetchTableStates return the > value or get it as an argument to the function and the global > variables can be removed. > +static List *table_states_not_ready = NIL; >
But we do update the states in the list table_states_not_ready in function process_syncing_tables_for_apply. So, the current arrangement looks good to me. -- With Regards, Amit Kapila.
