| From: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com> |
|---|---|
| To: | Arthur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Oleg Bartunov <obartunov(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, David Steele <david(at)pgmasters(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [HACKERS] [PATCH] Generic type subscripting |
| Date: | 2018-01-30 15:47:39 |
| Message-ID: | CA+q6zcUPeGhtNDqXo65o87Zfjjs0T5E0=+ab3nax9VXKch6LXw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 29 January 2018 at 14:41, Arthur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru> wrote:
>
> I have a few comments.
Thanks for suggestions, I've incorporated all of them in a new version of the
patch.
> SbsRoutines is not good name for me. SubscriptRoutines or
> SubscriptingRoutines sound better and it is consistent with other
> structures.
In general I agree, and I thought about that while implementing this structure.
But my concern is that some parts of subscripting infrastructure are annoyingly
verbose, maybe it makes sense to come up with a short abbreviation and use it
everywhere across this code.
| Attachment | Content-Type | Size |
|---|---|---|
| 0005-Subscripting-documentation-v7.patch | text/x-patch | 18.7 KB |
| 0004-Subscripting-for-jsonb-v7.patch | text/x-patch | 32.9 KB |
| 0003-Subscripting-for-array-v7.patch | text/x-patch | 12.8 KB |
| 0002-Base-implementation-of-subscripting-mechanism-v7.patch | text/x-patch | 127.0 KB |
| 0001-Renaming-for-new-subscripting-mechanism-v7.patch | text/x-patch | 44.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2018-01-30 15:59:32 | Re: [HACKERS] MERGE SQL Statement for PG11 |
| Previous Message | Robert Haas | 2018-01-30 14:35:50 | Re: [HACKERS][PATCH] Applying PMDK to WAL operations for persistent memory |