patch review "split string to table"

From: Srinivas Bethi <sbethi(at)outlook(dot)com>
To: "pgadmin-hackers(at)lists(dot)postgresql(dot)org" <pgadmin-hackers(at)lists(dot)postgresql(dot)org>
Subject: patch review "split string to table"
Date: 2020-07-01 15:15:43
Message-ID: BY5PR11MB4119F8D25E2779B00002F2C0DD6C0@BY5PR11MB4119.namprd11.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers


Hi Pavel,

I and Taylor Vesely (Greenplum) reviewed the patch "split string to table". as part of the June patch review workshop.

Thanks

It looks like you removed some unrelated documentation:



| @@ -16717,21 +16738,6 @@ strict $.track.segments[*].location | </para></entry> | </row> | | - <row>
| - <entry role="func_table_entry"><para role="func_signature">

| - <replaceable>string</replaceable> <literal>starts with</literal> <replaceable>string</replaceable>

| - <returnvalue>boolean</returnvalue>

| - </para>

| - <para>

| - Tests whether the second operand is an initial substring of the first

| - operand.

| - </para>

| - <para>

| - <literal>jsonb_path_query('["John Smith", "Mary Stone", "Bob Johnson"]', '$[*] ? (@ starts with "John")')</literal>

| - <returnvalue>"John Smith"</returnvalue>

| - </para></entry>

| - </row>

| -


We didn't actually confirm the performance benefits with un_nest()

Browse pgadmin-hackers by date

  From Date Subject
Next Message Pradip Parkale 2020-07-02 05:21:39 Re: [pgAdmin][RM5516]: Added support of Row Security Policies
Previous Message Akshay Joshi 2020-07-01 12:02:33 pgAdmin 4 commit: Fixed database connection issue, regression of SonarQ