On Fri, Dec 2, 2022 at 4:57 PM Hayato Kuroda (Fujitsu) <kuroda.hay...@fujitsu.com> wrote: > > handle_streamed_transaction() > > ``` > + if (apply_action == TRANS_LEADER_SEND_TO_PARALLEL) > + pa_send_data(winfo, s->len, s->data); > + else > + stream_write_change(action, &original_msg); > ``` > > Comments are needed here, 0001 has that bu removed in 0002. > There are some similar lines. >
I have suggested removing it because they were just saying what is evident from the code and doesn't seem to be adding any value. I would say they were rather confusing. -- With Regards, Amit Kapila.