On Tue, Mar 16, 2021 at 6:22 PM vignesh C <[email protected]> wrote: > > On Mon, Mar 15, 2021 at 6:14 PM Ajin Cherian <[email protected]> wrote: > > > > On Mon, Mar 15, 2021 at 2:04 PM Amit Kapila <[email protected]> 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.
