Thread

Commits

  1. Remove mention of the version number from pg_trgm docs

  1. Mention of PG 14 in docs

    Bruce Momjian <bruce@momjian.us> — 2021-05-03T23:49:14Z

    In patch "Handle equality operator in contrib/pg_trgm" (935f66665), I
    see this text:
    
    -->	<literal>~</literal> and <literal>~*</literal> queries.  Beginning in
    -->	<productname>PostgreSQL</productname> 14, these indexes also support
    	equality operator (simple comparison operators are not supported).
    	Note that those indexes may not be as efficient as regular B-tree indexes
    	for equality operator.
    
    As far as I know, we don't usually mention what version _added_ a
    feature like this.  I suggest the PG 14 mention be removed.
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        https://momjian.us
      EDB                                      https://enterprisedb.com
    
      If only the physical world exists, free will is an illusion.
    
    
    
    
    
  2. Re: Mention of PG 14 in docs

    Bruce Momjian <bruce@momjian.us> — 2021-05-04T00:10:24Z

    On Mon, May  3, 2021 at 07:49:14PM -0400, Bruce Momjian wrote:
    > In patch "Handle equality operator in contrib/pg_trgm" (935f66665), I
    > see this text:
    > 
    > -->	<literal>~</literal> and <literal>~*</literal> queries.  Beginning in
    > -->	<productname>PostgreSQL</productname> 14, these indexes also support
    > 	equality operator (simple comparison operators are not supported).
    > 	Note that those indexes may not be as efficient as regular B-tree indexes
    > 	for equality operator.
    > 
    > As far as I know, we don't usually mention what version _added_ a
    > feature like this.  I suggest the PG 14 mention be removed.
    
    Also, what does "simple comparison operators are not supported" mean?
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        https://momjian.us
      EDB                                      https://enterprisedb.com
    
      If only the physical world exists, free will is an illusion.
    
    
    
    
    
  3. Re: Mention of PG 14 in docs

    Bruce Momjian <bruce@momjian.us> — 2021-05-04T00:20:46Z

    On Mon, May  3, 2021 at 08:10:24PM -0400, Bruce Momjian wrote:
    > On Mon, May  3, 2021 at 07:49:14PM -0400, Bruce Momjian wrote:
    > > In patch "Handle equality operator in contrib/pg_trgm" (935f66665), I
    > > see this text:
    > > 
    > > -->	<literal>~</literal> and <literal>~*</literal> queries.  Beginning in
    > > -->	<productname>PostgreSQL</productname> 14, these indexes also support
    > > 	equality operator (simple comparison operators are not supported).
    > > 	Note that those indexes may not be as efficient as regular B-tree indexes
    > > 	for equality operator.
    > > 
    > > As far as I know, we don't usually mention what version _added_ a
    > > feature like this.  I suggest the PG 14 mention be removed.
    > 
    > Also, what does "simple comparison operators are not supported" mean?
    
    Please ignore this second question since is has been fixed by:
    
    	Author: Alexander Korotkov <akorotkov@postgresql.org>
    	2020-11-16 [7adb8feb0] Reword 'simple comparison' => 'inequality' in pgtrgm.sgm
    	
    	Reword 'simple comparison' => 'inequality' in pgtrgm.sgml
    	
    	Discussion: https://postgr.es/m/CAMkU%3D1zxfA8_MGBW6sJMj54p8nPoe4bMb5LoG-rMYZVPq4j08Q%40mail.gmail.com
    	Author: Jeff Janes
    
    The first question about PG 14 is still valid.
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        https://momjian.us
      EDB                                      https://enterprisedb.com
    
      If only the physical world exists, free will is an illusion.
    
    
    
    
    
  4. Re: Mention of PG 14 in docs

    Alexander Korotkov <aekorotkov@gmail.com> — 2021-05-04T00:51:51Z

    Hi, Bruce!
    
    On Tue, May 4, 2021 at 2:49 AM Bruce Momjian <bruce@momjian.us> wrote:
    > In patch "Handle equality operator in contrib/pg_trgm" (935f66665), I
    > see this text:
    >
    > -->     <literal>~</literal> and <literal>~*</literal> queries.  Beginning in
    > -->     <productname>PostgreSQL</productname> 14, these indexes also support
    >         equality operator (simple comparison operators are not supported).
    >         Note that those indexes may not be as efficient as regular B-tree indexes
    >         for equality operator.
    >
    > As far as I know, we don't usually mention what version _added_ a
    > feature like this.  I suggest the PG 14 mention be removed.
    
    I agree with this.  Thank you for pointing.  Are you OK with the attached patch?
    
    ------
    Regards,
    Alexander Korotkov
    
  5. Re: Mention of PG 14 in docs

    Bruce Momjian <bruce@momjian.us> — 2021-05-04T00:55:56Z

    On Tue, May  4, 2021 at 03:51:51AM +0300, Alexander Korotkov wrote:
    > Hi, Bruce!
    > 
    > On Tue, May 4, 2021 at 2:49 AM Bruce Momjian <bruce@momjian.us> wrote:
    > > In patch "Handle equality operator in contrib/pg_trgm" (935f66665), I
    > > see this text:
    > >
    > > -->     <literal>~</literal> and <literal>~*</literal> queries.  Beginning in
    > > -->     <productname>PostgreSQL</productname> 14, these indexes also support
    > >         equality operator (simple comparison operators are not supported).
    > >         Note that those indexes may not be as efficient as regular B-tree indexes
    > >         for equality operator.
    > >
    > > As far as I know, we don't usually mention what version _added_ a
    > > feature like this.  I suggest the PG 14 mention be removed.
    > 
    > I agree with this.  Thank you for pointing.  Are you OK with the attached patch?
    
    Yes, sure.
    
    -- 
      Bruce Momjian  <bruce@momjian.us>        https://momjian.us
      EDB                                      https://enterprisedb.com
    
      If only the physical world exists, free will is an illusion.
    
    
    
    
    
  6. Re: Mention of PG 14 in docs

    Alexander Korotkov <aekorotkov@gmail.com> — 2021-05-04T01:01:40Z

    On Tue, May 4, 2021 at 3:55 AM Bruce Momjian <bruce@momjian.us> wrote:
    > On Tue, May  4, 2021 at 03:51:51AM +0300, Alexander Korotkov wrote:
    > > Hi, Bruce!
    > >
    > > On Tue, May 4, 2021 at 2:49 AM Bruce Momjian <bruce@momjian.us> wrote:
    > > > In patch "Handle equality operator in contrib/pg_trgm" (935f66665), I
    > > > see this text:
    > > >
    > > > -->     <literal>~</literal> and <literal>~*</literal> queries.  Beginning in
    > > > -->     <productname>PostgreSQL</productname> 14, these indexes also support
    > > >         equality operator (simple comparison operators are not supported).
    > > >         Note that those indexes may not be as efficient as regular B-tree indexes
    > > >         for equality operator.
    > > >
    > > > As far as I know, we don't usually mention what version _added_ a
    > > > feature like this.  I suggest the PG 14 mention be removed.
    > >
    > > I agree with this.  Thank you for pointing.  Are you OK with the attached patch?
    >
    > Yes, sure.
    
    Perfect, pushed.
    
    ------
    Regards,
    Alexander Korotkov