On Mon, Oct 21, 2013 at 11:12 PM, Alexander Korotkov
<aekorot...@gmail.com>wrote:

> Attached version of patch is debugged. I would like to note that number of
> bugs was low and it wasn't very hard to debug. I've rerun tests on it. You
> can see that numbers are improved as the result of your refactoring.
>
>          event         |     period
> -----------------------+-----------------
>  index_build           | 00:01:45.416822
>  index_build_recovery  | 00:00:06
>  index_update          | 00:05:17.263606
>  index_update_recovery | 00:01:22
>  search_new            | 00:24:07.706482
>  search_updated        | 00:26:25.364494
> (6 rows)
>
>      label      | blocks_mark
> ----------------+-------------
>  search_new     |   847587636
>  search_updated |   881210486
> (2 rows)
>
>      label     |   size
> ---------------+-----------
>  new           | 419299328
>  after_updates | 715915264
> (2 rows)
>
> Beside simple bugs, there was a subtle bug in incremental item indexes
> update. I've added some more comments including ASCII picture about how
> item indexes are shifted.
>
> Now, I'm trying to implement support of old page format. Then we can
> decide which approach to use.
>

Attached version of patch has support of old page format. Patch still needs
more documentation and probably refactoring, but I believe idea is clear
and it can be reviewed. In the patch I have to revert change of null
category placement for compatibility with old posting list format.

------
With best regards,
Alexander Korotkov.

Attachment: gin-packed-postinglists-12.patch.gz
Description: GNU Zip compressed data

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to