Re: [PATH] Jsonb, insert a new value into an array at arbitrary position
| От | Vitaly Burovoy |
|---|---|
| Тема | Re: [PATH] Jsonb, insert a new value into an array at arbitrary position |
| Дата | |
| Msg-id | 20160331155621.1315.75078.pgcf@coridan.postgresql.org обсуждение |
| Ответ на | Re: [PATH] Jsonb, insert a new value into an array at arbitrary position (Dmitry Dolgov <9erthalion6@gmail.com>) |
| Список | pgsql-hackers |
The following review has been posted through the commitfest application: make installcheck-world: tested, passed Implements feature: tested, passed Spec compliant: not tested Documentation: tested, passed I have reviewed this patch. It applies cleanly at the top of current master (3501f71), compiles silently and implements declared feature. The documentation describes behavior of the function with pointing to a difference between inserting into JsonbObject andJsonbArray. The code is clean and commented. Linked comment is changed too. Regression tests cover possible use cases and edge cases. Notes for a committer: 1. pg_proc.h has changed, so the CATALOG_VERSION_NO must also be changed. 2. Code comments and changes in the documentation need proof-reading by a native English speaker, which the Author and I are not. The new status of this patch is: Ready for Committer
В списке pgsql-hackers по дате отправления: