Re: pgsql: Allow to omit boundaries in array subscript
От | Teodor Sigaev |
---|---|
Тема | Re: pgsql: Allow to omit boundaries in array subscript |
Дата | |
Msg-id | 5674503A.1050301@sigaev.ru обсуждение исходный текст |
Ответ на | Re: pgsql: Allow to omit boundaries in array subscript (Tom Lane <tgl@sss.pgh.pa.us>) |
Список | pgsql-committers |
I will work on it. Tom Lane wrote: > Teodor Sigaev <teodor@sigaev.ru> writes: >> Allow to omit boundaries in array subscript > > For the record, this patch wasn't nearly ready to commit. > > * User-facing documentation is very poor; at the least it needs > copy-editing by someone with better command of English. > > * Comments are next to nonexistent. I note for example that the > description of struct A_Indices wasn't updated, much less that of > ArrayRef. > > * Some error messages aren't meeting the style guidelines. > > * I'm dubious that the parsetree representation is well-chosen. > Probably a single is_slice flag would have been better. > > * Dumps core on fixed-length arrays, eg > > regression=# select f1[1:] from point_tbl; > server closed the connection unexpectedly > > I think it should be reverted and sent back for another round of > review. If not, I'll probably spend tomorrow cleaning it up, > but I really have other things I should be doing. > > regards, tom lane > > -- Teodor Sigaev E-mail: teodor@sigaev.ru WWW: http://www.sigaev.ru/
В списке pgsql-committers по дате отправления: