|From:||Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>|
|To:||John Naylor <john(dot)naylor(at)2ndquadrant(dot)com>, Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com>|
|Cc:||Mithun Cy <mithun(dot)cy(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: WIP: Avoid creation of the free space map for small tables|
|Views:||Raw Message | Whole Thread | Download mbox|
On Tue, Jan 29, 2019 at 8:12 PM John Naylor <john(dot)naylor(at)2ndquadrant(dot)com> wrote:
> On Tue, Jan 29, 2019 at 11:56 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> > You can find this change in attached patch. Then, I ran the make
> > check in src/bin/pgbench multiple times using test_conc_insert.sh.
> > You can vary the number of times the test should run, if you are not
> > able to reproduce it with this.
> > The attached patch (clear_local_map_if_exists_1.patch) atop the main
> > patch fixes the issue for me. Kindly verify the same.
> I got one failure in 50 runs. With the new patch, I didn't get any
> failures in 300 runs.
Thanks for verification. I have included it in the attached patch and
I have also modified the page.sql test to have enough number of pages
in relation so that FSM will get created irrespective of alignment
boundaries. Masahiko San, can you verify if this now works for you?
There are two more failures which we need to something about.
1. Make fsm.sql independent of vacuum without much losing on coverage
of newly added code. John, I guess you have an idea, see if you can
take care of it, otherwise, I will see what I can do for it.
2. I still could not figure out how to verify if the failure on Jacana
will be fixed. I have posted some theory above and the attached patch
has a solution for it, but I think it would be better if find out some
way to verify the same.
Note - you might see some cosmetic changes in freespace.c due to pgindent.
|Next Message||Tom Lane||2019-01-30 03:42:34||Re: Early WIP/PoC for inlining CTEs|
|Previous Message||Alvaro Herrera||2019-01-30 03:25:53||Re: speeding up planning with partitions|