From: | Jasmin Dizdarevic <jasmin(dot)dizdarevic(at)gmail(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Debug works, Release not |
Date: | 2011-01-07 18:28:21 |
Message-ID: | AANLkTinytx84tZ=Sh7LjyKA2jk=CHqTcYNkaf-VR2DHS@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
2011/1/7 Dave Page <dpage(at)pgadmin(dot)org>
> On Thu, Jan 6, 2011 at 7:30 PM, Jasmin Dizdarevic
> <jasmin(dot)dizdarevic(at)gmail(dot)com> wrote:
> > Hi,
> > here is the patch, that enables UI for pgAgent 4.
>
> Cool!
>
> > It would be great if somebody could try to build in Release mode.
> > The UI should work with pgAgent schema version 3 - if not, please tell
> me.
>
> It doesn't. See the attached screenshot for the error seen upon
> connection. Further errors when checking opening the new job dialogue,
> then a crash. It shouldn't take much effort to figure out I imagine
> :-)
>
> Great that you've tested it on Mac OS. I've used an uninitialized
int-variable that doesn't behaves like on windows. that's the reason for the
error's. It's already fixed.
> > Please tell me also, if I have violated coding policies or something
> else.
>
> Actually, it looks pretty clean to me. A good first-ever patch, style
> wise and architecturally.
>
> Thanks, pgAdmin's architecture is clean and it's pretty easy to start with
it.
> > To test the new features, you can apply attached pgagent_upgrade.sql.
>
> You're missing a ; in there on line 19, and have terminated lines 27 -
> 29 with , instead of ;.
>
> OK.
> Fixing that, and I can see the new UI without any errors, however, I
> cannot test it without an agent. Actually, that's pretty annoying -
> currently I can configure my jobs before I setup my agent, but with
> this change, I can no longer do that, because the agents need to have
> registered their email capability in the database already. I'm not
> sure that's behaviour that is desirable... but it becomes irrelevant
> if we fully integrate the SMTP code as discussed.
>
> I'm still waiting for the authorization :(
Regards
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2011-01-09 14:41:46 | Re: pgAdmin III commit: Pull keywords dynamically from PostgreSQL headers |
Previous Message | Dave Page | 2011-01-07 14:36:08 | Re: Debug works, Release not |