| From: | Jeff Davis <pgsql(at)j-davis(dot)com> |
|---|---|
| To: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Corey Huinker <corey(dot)huinker(at)gmail(dot)com> |
| Subject: | Re: Statistics import and export: difference in statistics of materialized view dumped |
| Date: | 2025-03-28 23:20:58 |
| Message-ID: | 29a2087d79080e9bbdde6d2423a94635cca639b8.camel@j-davis.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, 2025-03-28 at 14:53 +0530, Ashutosh Bapat wrote:
> When I applied v1 it didn't pass.
I applied v1 on top of master as of March 15 (771ba90298), and then
took your two changes adding the tests, and it passed.
Version v2j is just rebased forward, which involved a trivial merge
conflict.
> But applying v2j-0003, the test passes.
Great.
> It feels like we can simplify this as: REFRESH MATERIALIZED VIEW
> replaces storage and resets the stats hence the Matview stats should
> be printed after printing REFRESH MATERIALIZED VIEW command in
> SECTION_POST_DATA. Also the materialized view ... . Hence the section
> may be updated later for materialized views.
Done, with a few modifications. Hopefully it's simpler.
Tom suggested that there may be other dependency problems lurking, but
I believe the patch is an improvement, so I committed it now to unblock
your work.
Regards,
Jeff Davis
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Masahiko Sawada | 2025-03-29 00:01:14 | Re: On non-Windows, hard depend on uselocale(3) |
| Previous Message | Euler Taveira | 2025-03-28 23:09:25 | wal writer: obsolete sentence |