[prev in list] [next in list] [prev in thread] [next in thread] 

List:       postgresql-general
Subject:    Re: [HACKERS] [PATCH] few fts functions for jsonb
From:       Oleg Bartunov <obartunov () gmail ! com>
Date:       2017-03-31 19:17:52
Message-ID: CAF4Au4wAxSOcUN41uVO03CAQ_P5bvM_ELMmi9BHoxBLQtiAw2Q () mail ! gmail ! com
[Download RAW message or body]

On 30 Mar 2017 23:43, "Dmitry Dolgov" <9erthalion6@gmail.com> wrote:

On 31 March 2017 at 00:01, Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
wrote:
>
> I have just noticed as I was writing/testing the non-existent docs for
> this patch that it doesn't supply variants of to_tsvector that take a
> regconfig as the first argument. Is there a reason for that? Why
> should the json(b) versions be different from the text versions?

No, there is no reason, I just missed that. Here is a new version of the
patch (only the functions part)
to add those variants.


Congratulations with patch committed, who will write an addition
documentation? I think we need to touch  FTS and JSON parts.

[Attachment #3 (text/html)]

<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 \
Mar 2017 23:43, &quot;Dmitry Dolgov&quot; &lt;<a href="mailto:9erthalion6@gmail.com" \
target="_blank">9erthalion6@gmail.com</a>&gt; wrote:<br \
type="attribution"><blockquote class="m_-2819040210173539784quote" style="margin:0 0 \
0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div \
class="gmail_extra"><div class="gmail_quote"><div \
class="m_-2819040210173539784quoted-text">On 31 March 2017 at 00:01, Andrew Dunstan \
&lt;<a href="mailto:andrew.dunstan@2ndquadrant.com" \
target="_blank">andrew.dunstan@2ndquadrant.co<wbr>m</a>&gt; wrote: <br>&gt;<br>&gt; I \
have just noticed as I was writing/testing the non-existent docs for<br>&gt; this \
patch that it doesn&#39;t supply variants of to_tsvector that take a<br>&gt; \
regconfig as the first argument. Is there a reason for that? Why<br>&gt; should the \
json(b) versions be different from the text versions?<br><br></div>No, there is no \
reason, I just missed that. Here is a new version of the patch (only the functions \
part)</div><div class="gmail_quote">to add those variants.</div></div></div> \
</blockquote></div><br></div></div><div class="gmail_extra" \
dir="auto">Congratulations with patch committed, who will write an addition \
documentation? I think we need to touch   FTS and JSON parts.</div></div>



[prev in list] [next in list] [prev in thread] [next in thread]