On Wed, Jul 17, 2019 at 01:31:05PM +0200, Daniel Verite wrote: > The tab width should be 4. Please have a look at > https://www.postgresql.org/docs/current/source-format.html > It also explains why opportunistic reformatting is futile, anyway:
- char *schema, - *table, - *field; + char *schema, + *table, + *field; The patch has some noise. For something of this size, I don't think that it is an issue though ;) > It might be useful to display the progress report in the loop, but > it appears that even when there's nothing to remove, vacuumlo is > likely to take a long time, because of the method it uses: > > [stuff] > > That might be much faster than #1 and #2, especially in the case when > there's only one SELECT in that subquery and no UNION ALL is even > necessary. Sure. However do we need to introduce this much complication as a goal for this patch though whose goal is just to provide hints about the progress of the work done by vacuumlo? I have just looked at the latest patch and the thing is actually much more simple than what I recalled. One comment I have is if we should also report in the progress not only the percentage, but also the raw numbers of deleted entries with the total numbers of entries to delete. Timur, what do you think? -- Michael
signature.asc
Description: PGP signature
