<p dir="ltr">Hi Amit,<p dir="ltr">Thank you for reviewing the patch.<br /> On Jan 26, 2016 9:51 AM, "Amit Langote"
<<ahref="mailto:Langote_Amit_f8@lab.ntt.co.jp">Langote_Amit_f8@lab.ntt.co.jp</a>> wrote:<br /> ><br /> ><br
/>> Hi Vinayak,<br /> ><br /> > On 2016/01/25 20:58, Vinayak Pokale wrote:<br /> > > Hi,<br /> >
><br/> > > Please find attached updated patch with an updated interface.<br /> > ><br /> ><br /> >
Thanksfor updating the patch.<br /> ><br /> > > I added the below interface to update the<br /> > >
scanned_heap_pages,scanned_index_pagesand index_scan_count only.<br /> > > void
pgstat_report_progress_scanned_pages(intnum_of_int, uint32<br /> > > *progress_scanned_pages_param)<br /> ><br
/>> I think it's still the same interface with the names changed. IIRC, what<br /> > was suggested was to provide
away to not have to pass the entire array<br /> > for the update of a single member of it. Just pass the index of
the<br/> > updated member and its new value. Maybe, something like:<br /> ><br /> > void
pgstat_progress_update_counter(intindex, uint32 newval);<br /> ><br /> > The above function would presumably
updatethe value of<br /> > beentry.st_progress_counter[index] or something like that.<p dir="ltr">Understood. I will
updatethe patch.<p dir="ltr">Regards,<br /> Vinayak<br />