On Sat, Jul 6, 2019 at 12:05 AM Amit Kapila <amit.kapil...@gmail.com> wrote:
> On Tue, Jul 2, 2019 at 1:00 AM Ashwin Agrawal <aagra...@pivotal.io> wrote: > > Please find attached v2 of patch 1 without objectionable comment change. > v1 of patch 2 attaching here just for convenience, no modifications made to > it. > > > > 0001* > * See table_index_fetch_tuple's comment about what the difference between > - * these functions is. This function is the correct to use outside of > - * index entry->table tuple lookups. > + * these functions is. This function is correct to use outside of index > + * entry->table tuple lookups. > > How about if we write the last line of comment as "It is correct to > use this function outside of index entry->table tuple lookups."? I am > not an expert on this matter, but I find the way I am suggesting > easier to read. > I am fine with the way you have suggested.