From: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, Amit Langote <amitlangote09(at)gmail(dot)com>, pokurev(at)pm(dot)nttdata(dot)co(dot)jp, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, bannos(at)nttdata(dot)co(dot)jp |
Subject: | Re: [PROPOSAL] VACUUM Progress Checker. |
Date: | 2016-03-10 05:29:50 |
Message-ID: | 56E1064E.9010908@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2016/03/10 2:16, Robert Haas wrote:
> On Wed, Mar 9, 2016 at 2:37 AM, Amit Langote
> <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>> On 2016/03/09 10:11, Amit Langote wrote:
>>> The attached revision addresses above and one of Horiguchi-san's comments
>>> in his email yesterday.
>>
>> I fixed one more issue in 0002 per Horiguchi-san's comment. Sorry about
>> so many versions.
>
> I've committed 0001 with heavy revisions. Just because we don't need
> an SQL-visible function to clear the command progress doesn't mean we
> don't need to clear it at all; rather, it has to happen automatically.
> I also did a bunch of identifier renaming, added datid to the view
> output, adjusted the comments, and so on. Please rebase the remainder
> of the series. Thanks.
Great, thanks a lot for the review and committing in much better shape!
I rebased remainder patches (attached).
0001 is a small patch to fix issues reported by Tomas and Vinayak. 0002
and 0003 are WIP patches to implement progress reporting for vacuum.
Thanks,
Amit
Attachment | Content-Type | Size |
---|---|---|
0001-Some-minor-fixes-in-commit-b6fb6471.patch | text/x-diff | 1.7 KB |
0002-WIP-Implement-progress-reporting-for-VACUUM-command-v7.patch | text/x-diff | 12.9 KB |
0003-WIP-Add-a-block-number-argument-to-index-bulk-delete-v7.patch | text/x-diff | 10.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2016-03-10 06:24:06 | Re: BUG #13755: pgwin32_is_service not checking if SECURITY_SERVICE_SID is disabled |
Previous Message | Amit Kapila | 2016-03-10 05:18:41 | Re: RFC: replace pg_stat_activity.waiting with something more descriptive |