Re: GIN - Generalized Inverted iNdex. Try 2. - Mailing list pgsql-hackers
From | Teodor Sigaev |
---|---|
Subject | Re: GIN - Generalized Inverted iNdex. Try 2. |
Date | |
Msg-id | 444F9767.2040605@sigaev.ru Whole thread Raw |
In response to | Re: GIN - Generalized Inverted iNdex. Try 2. (Jeremy Drake <pgsql@jdrake.com>) |
List | pgsql-hackers |
> I just built this, and noticed that the regression test for "opr_sanity" > fails with your patch. I attached the regression.diffs. Sorry, this part isn't done yet, because we are waiting of community decision.. We don't add regression test yet. If community don't like to include GIN in core, we make a contrib/gin, but in this case GIN can't use WAL feature because of WAL interface can't call user-defined function. The reason for first diff is a hardcoded 'gist' index: -- We have to exclude GiST, unfortunately, since it hasn't got anyfixed -- requirements about strategy operators. SELECT p1.oid, p1.amname, p2.oid, p2.opcname, p3.amopsubtype FROM pg_am AS p1, pg_opclass AS p2, pg_amop AS p3 WHERE p2.opcamid = p1.oid AND p3.amopclaid = p2.oid AND p1.amname != 'gist' AND p1.amstrategies != (SELECT count(*)FROM pg_amop AS p4 WHERE p4.amopclaid = p2.oid AND p4.amopsubtype= p3.amopsubtype); Second is right diff. For the thread one reason is that operations &&, ~, @ defined for anyarray, but used for particular types. > > > ------------------------------------------------------------------------ > > *** ./expected/opr_sanity.out Wed Jan 25 18:35:51 2006 > --- ./results/opr_sanity.out Wed Apr 26 08:31:13 2006 > *************** > *** 778,785 **** > WHERE p4.amopclaid = p2.oid AND > p4.amopsubtype = p3.amopsubtype); > oid | amname | oid | opcname | amopsubtype > ! -----+--------+-----+---------+------------- > ! (0 rows) > > -- Check that amopopr points at a reasonable-looking operator, ie a binary > -- operator yielding boolean. > --- 778,791 ---- > WHERE p4.amopclaid = p2.oid AND > p4.amopsubtype = p3.amopsubtype); > oid | amname | oid | opcname | amopsubtype > ! ------+--------+------+-----------+------------- > ! 2742 | gin | 2745 | _int4_ops | 0 > ! 2742 | gin | 2745 | _int4_ops | 0 > ! 2742 | gin | 2745 | _int4_ops | 0 > ! 2742 | gin | 2746 | _text_ops | 0 > ! 2742 | gin | 2746 | _text_ops | 0 > ! 2742 | gin | 2746 | _text_ops | 0 > ! (6 rows) > > -- Check that amopopr points at a reasonable-looking operator, ie a binary > -- operator yielding boolean. > *************** > *** 825,831 **** > 783 | 10 | <<| > 783 | 11 | |>> > 783 | 12 | |&> > ! (24 rows) > > -- Check that all operators linked to by opclass entries have selectivity > -- estimators. This is not absolutely required, but it seems a reasonable > --- 831,840 ---- > 783 | 10 | <<| > 783 | 11 | |>> > 783 | 12 | |&> > ! 2742 | 1 | && > ! 2742 | 2 | @ > ! 2742 | 3 | ~ > ! (27 rows) > > -- Check that all operators linked to by opclass entries have selectivity > -- estimators. This is not absolutely required, but it seems a reasonable > *************** > *** 847,854 **** > WHERE p1.amopopr = p2.oid AND p1.amopclaid = p3.oid AND > NOT binary_coercible(p3.opcintype, p2.oprleft); > amopclaid | amopopr | oid | oprname | opcname > ! -----------+---------+-----+---------+--------- > ! (0 rows) > > SELECT p1.amopclaid, p1.amopopr, p2.oid, p2.oprname, p3.opcname > FROM pg_amop AS p1, pg_operator AS p2, pg_opclass AS p3 > --- 856,869 ---- > WHERE p1.amopopr = p2.oid AND p1.amopclaid = p3.oid AND > NOT binary_coercible(p3.opcintype, p2.oprleft); > amopclaid | amopopr | oid | oprname | opcname > ! -----------+---------+------+---------+----------- > ! 2746 | 2750 | 2750 | && | _text_ops > ! 2745 | 2750 | 2750 | && | _int4_ops > ! 2746 | 2751 | 2751 | @ | _text_ops > ! 2745 | 2751 | 2751 | @ | _int4_ops > ! 2746 | 2752 | 2752 | ~ | _text_ops > ! 2745 | 2752 | 2752 | ~ | _int4_ops > ! (6 rows) > > SELECT p1.amopclaid, p1.amopopr, p2.oid, p2.oprname, p3.opcname > FROM pg_amop AS p1, pg_operator AS p2, pg_opclass AS p3 > > ====================================================================== > > > > ------------------------------------------------------------------------ > > > ---------------------------(end of broadcast)--------------------------- > TIP 5: don't forget to increase your free space map settings -- Teodor Sigaev E-mail: teodor@sigaev.ru WWW: http://www.sigaev.ru/
pgsql-hackers by date: