Re: [HACKERS] Show INHERIT in \du

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [HACKERS] Show INHERIT in \du
Дата
Msg-id 25919.1206055610@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [HACKERS] Show INHERIT in \du  ("Brendan Jurd" <direvus@gmail.com>)
Ответы Re: [HACKERS] Show INHERIT in \du  ("Brendan Jurd" <direvus@gmail.com>)
Список pgsql-patches
"Brendan Jurd" <direvus@gmail.com> writes:
> I've done up a patch per Tom's idea of combining the binary role
> attributes into a single column.

I started to look at committing this and realized that there's a very
nasty problem: our current approach to localizing the strings won't
work.  See this patch for background:
http://archives.postgresql.org/pgsql-committers/2007-12/msg00187.php

The code is now set up so that it can pass an entire field value
through gettext(), but if gettext recognizes the strings "foo" and
"bar" that doesn't mean it will do anything useful with "foo\nbar",
which is what this patch would require.

I suspect that to solve this in a non-kluge fashion we'd need to make
\du pull over the plain boolean and integer values, then build a new
PGresult data structure on its own.  Ugh.  (Actually, without any
support from libpq for building PGresults, it's hard to imagine doing
that in a way that wouldn't be a kluge itself.)

Or we could go back to the drawing board on what the output ought to
look like.

Thoughts?

            regards, tom lane

В списке pgsql-patches по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: [GENERAL] Empty arrays with ARRAY[]
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: [HACKERS] Show INHERIT in \du