| From: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com> |
|---|---|
| To: | Vitaly Burovoy <vitaly(dot)burovoy(at)gmail(dot)com> |
| Cc: | David Steele <david(at)pgmasters(dot)net>, Petr Jelinek <petr(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PATH] Jsonb, insert a new value into an array at arbitrary position |
| Date: | 2016-03-25 17:47:48 |
| Message-ID: | CA+q6zcVDZAJ51POUSTYdX+87hnZ9VFtkkvMRVWc4fOLhJ3pC9g@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Here is a new version of path, I hope I didn't miss anything. Few notes:
> 4.
> or even create a new constant (there can be better name for it):
> #define JB_PATH_CREATE_OR_INSERT (JB_PATH_INSERT_BEFORE |
> JB_PATH_INSERT_AFTER | JB_PATH_CREATE)
Good idea, thanks.
> 5.
> > if (op_type != JB_PATH_DELETE)
Yes, I just missed that in previous patch.
> 7.
> Please, return the "skip" comment.
Well, I'm still not so sure about that, but if you're so confident then ok
=)
| Attachment | Content-Type | Size |
|---|---|---|
| jsonb_insert_v4.patch | text/x-patch | 18.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Abhijit Menon-Sen | 2016-03-25 17:57:20 | Re: dealing with extension dependencies that aren't quite 'e' |
| Previous Message | Dilip Kumar | 2016-03-25 17:32:11 | Re: Move PinBuffer and UnpinBuffer to atomics |