> -----Original Message-----
> From: Andreas Pflug [mailto:pgadmin@pse-consulting.de]
> Sent: 22 June 2004 17:10
> To: Dave Page
> Cc: pgadmin-hackers@postgresql.org
> Subject: Misc topics
>
> Dave Page wrote:
>
> >
> >
> >I see what's happening. I dump things in text format more often than
> >not, and that's what it's barfing on. I think that needs to
> be handled
> >a lttle more cleanly when we release - perhaps check the file format
> >before passing it to pg_restore,
> >
> Ok, checking the file signature seems reasonable.
>
> > and if text, just load the first hundred lines or so for inspection.
> >
> >
> No. Advice to use Query Tool instead or sth like that.
Not really a good idea for say, a 300MB backup though is it?
> >Whilst we're on that subject, most of my backups have .sql
> extensions -
> >any objection to adding that as a defaul extension in the file open
> >dialogue.
> >
> >
> For backup only, not restore; we don't want to offer
> arbitrary scripts to pg_restore.
Mine are not arbitrary scripts. They are backups that happen to be in
plain text format, and have an appropriate .sql extension.
> >Finally, on a vaguely related note, I think we need to do some
> >re-factoring of the context and tools menus. Perhaps have a
> copy of the
> >tools menu as a sub menu of the context menu, rather than a
> seemingly
> >random inclusion of some items. Also, when items are active could do
> >with some though - for example, you cannot access the server status
> >when clicking a server node(!), only a database.
> >
> Well, if you insist we might enable server status on servers
> too :-) I admit we should have a look at the context menu. I
> deliberately left backup/restore out of it, because it's not
> so often in use, though it belongs there.
>
> Maybe we should *create* the context menu on-demand, instead
> of enabling/disabling. Disabled menus always signal "this
> item might be enabled under some circumstances", which is
> usually not true in that very context.
That's not a bad idea - the tools menu should remain constant of course.
Regards, Dave.
