Обсуждение: Crash after changing pgAgent Jobs

Hi Dhiraj, to this issue:

> One more behavior to the refresh-option below:
>
> If I just try to deactivate a Job on confirming I get the error-message:
> Cannot refresh browser
> There are properties dialogues open for one or more objects that would
> be refreshed. Please close the properties dialogues and try again.
>
> But I don't have any other dialogues open.
>

This happens if the refresh option in Options->UI Misellaneous->Refresh on click is set to
Refresh object and children on click

Sorry for my inaccurateness!
Günther


___________________________________
Günther Unterrainer
Wind Generators Projects & Service IT

LEITWIND AG - SpA
Brennerstraße 34 - Via Brennero, 34
I-39049 Sterzing - Vipiteno
Tel. +39 0472 722 443

http://www.leitwind.com

Please consider the environment before printing this email

Dhiraj Chawla <dhiraj.chawla@enterprisedb.com> wrote on 08.11.2012 14:21:16:

> Dhiraj Chawla <dhiraj.chawla@enterprisedb.com>
> 08.11.2012 14:21
>
> To
>
> UNTERRAINER.Guenther@leitwind.com, Dave Page <dpage@pgadmin.org>
>
> cc
>
> pgAdmin Support <pgadmin-support@postgresql.org>
>
> Subject
>
> Re: [pgadmin-support] Crash after changing pgAgent Jobs

>
> Hi Guenther,

> On Tue, Nov 6, 2012 at 5:12 PM, Dave Page <dpage@pgadmin.org> wrote:
> Dhiraj, can you look into this please?
>
> Thanks.
>
> On Mon, Nov 5, 2012 at 9:24 AM,  <UNTERRAINER.Guenther@leitwind.com> wrote:
> > One more behavior to the refresh-option below:
> >
> > If I just try to deactivate a Job on confirming I get the error-message:
> > Cannot refresh browser
> > There are properties dialogues open for one or more objects that would
> > be refreshed. Please close the properties dialogues and try again.
> >
> > But I don't have any other dialogues open.
> >
>
> I am not able to reproduce this case at my end with the steps as
> given by you. Can you please provide me more information and
> specific steps that you did to reproduce this error?
>
>  
> > Regards
> > Günther
> >
> >
> >
> >
> >
> > Hi all,
> >
> > Since I am using PgAdmin v.1.16.0 on Windows 7 64 bit when I change anything
> > on a pgAgent job
> > pgAgent chrashes.
> > This happens only if I the new tree refresh option is turned on in
> > Options->UI Miscellaneous->Refresh on click->Refresh object on click
> > When I turn this option off it works fine.
> >
>
> This crash was reproducible to me and am looking into it.
>  
> > Thank you!
> >
> > Günther
> >
> > --------------------------------------------------------------------------
> > This email and any files transmitted with it are confidential and intended
> > solely for the use of the individual or entity to whom they are addressed.
> > If you have received this email in error please notify the system manager.
> >
> > Think GREEN before you print!
> > Before you print, please consider if it is necessary.
> > Printing uses electricity, ink and paper.
> > --------------------------------------------------------------------------
>
>

> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

>
> --
> regards,
>
> Dhiraj Chawla
> Software Engineer
> EnterpriseDB Corporation
> The Enterprise PostgreSQL Company
>
> Phone: +91-20-30589522
--------------------------------------------------------------------------
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error please notify the system manager.

Think GREEN before you print!
Before you print, please consider if it is necessary.
Printing uses electricity, ink and paper.
--------------------------------------------------------------------------

Hi Dhiraj,

I tried it out again ('Refresh object and children on click' for Refresh on click is already set)
I have also selected the option to open properties of an object by double click.

So when I do:
1. open the properties dialog by double click
2. check (or uncheck, it happens always the same) the 'enabled' checkbox and do ok
3. Click OK button and the message box pops up.
4. If I confirm the error message and refresh the Jobs list the change is done

an other observation is that when I to the same but open the properties dialog by right mouse click -> properties,
then after confirming the same change as above I get two times the message box with the mentioned error,
but the change is done, as above

Thanks

Günther



___________________________________
Günther Unterrainer
Wind Generators Projects & Service IT

LEITWIND AG - SpA
Brennerstraße 34 - Via Brennero, 34
I-39049 Sterzing - Vipiteno
Tel. +39 0472 722 443

http://www.leitwind.com

Please consider the environment before printing this email

pgadmin-support-owner@postgresql.org wrote on 09.11.2012 12:26:02:

> Dhiraj Chawla <dhiraj.chawla@enterprisedb.com>
> Sent by: pgadmin-support-owner@postgresql.org
>
> 09.11.2012 12:26

>
> To
>
> UNTERRAINER.Guenther@leitwind.com, Dave Page <dpage@pgadmin.org>
>
> cc

>
> pgadmin-support@postgresql.org

>
> Subject
>
> Re: [pgadmin-support] Crash after changing pgAgent Jobs

>
> Hi Günther,

>
> I had a look at the issue and the case is reproducible to me also,
> but this happens only if do the following steps:
>
> 1. Open options dialog and in the UI Miscellaneous tab select
> 'Refresh object and children on click' for Refresh on click option.
> 2. Open the properties dialog of a job.
> 3. Select some other node on the object browser.
> 4. Then select again the job node.
> 5. The Message box with message  "There are properties dialogues
> open for one or more objects that would be refreshed. Please close
> the properties dialogues and try again.".
>
> From the understanding that I got after reading your mail what
> happens in your case is as follows:
>
> 1. Open options dialog and in the UI Miscellaneous tab select
> 'Refresh object and children on click' for Refresh on click option.
> 2. Open the properties dialog of a job.
> 3. Uncheck the 'Enabled' the checkbox in the dialog.
> 4. Click OK button and the message box pop-up.
>
> Well the same message box was not reproducible to me based on these steps.
>
> If the first set of steps are true then according to the code
> the behavior is expected, because when we refresh a node or it's
> children, we re-construct their objects and the previous objects are
> deleted. If we do not keep this check then this might lead to
> crashes and segmentation faults as the dialogs tend to use the pre-
> constructed objects.
>
> I am not sure if the text of the message box shown in this case is
> appropriate or not in context to the situation in which it gets pop-ed up.
>
> Dave, Thoughts/Comments?
>
> Thanks,
>
> On Fri, Nov 9, 2012 at 1:35 PM, <UNTERRAINER.Guenther@leitwind.com> wrote:
> Hi Dhiraj, to this issue:
>
>
> > One more behavior to the refresh-option below:
> >
> > If I just try to deactivate a Job on confirming I get the error-message:
> > Cannot refresh browser
> > There are properties dialogues open for one or more objects that would
> > be refreshed. Please close the properties dialogues and try again.
> >
> > But I don't have any other dialogues open.
> >

> This happens if the refresh option in Options->UI
> Misellaneous->Refresh on click is set to
> Refresh object and children on click
>
> Sorry for my inaccurateness!
> Günther
>
>
> ___________________________________
> Günther Unterrainer
> Wind Generators Projects & Service IT
>
> LEITWIND AG - SpA
> Brennerstraße 34 - Via Brennero, 34
> I-39049 Sterzing - Vipiteno
> Tel. +39 0472 722 443
>
> http://www.leitwind.com
>
> Please consider the environment before printing this email
>
> Dhiraj Chawla <dhiraj.chawla@enterprisedb.com> wrote on 08.11.2012 14:21:16:
>
> > Dhiraj Chawla <dhiraj.chawla@enterprisedb.com>
> > 08.11.2012 14:21
> >
> > To
> >
> > UNTERRAINER.Guenther@leitwind.com, Dave Page <dpage@pgadmin.org>
> >
> > cc
> >
> > pgAdmin Support <pgadmin-support@postgresql.org>
> >
> > Subject
> >
> > Re: [pgadmin-support] Crash after changing pgAgent Jobs
>
> >
> > Hi Guenther,
>
> > On Tue, Nov 6, 2012 at 5:12 PM, Dave Page <dpage@pgadmin.org> wrote:
> > Dhiraj, can you look into this please?
> >
> > Thanks.
> >
> > On Mon, Nov 5, 2012 at 9:24 AM,  <UNTERRAINER.Guenther@leitwind.com> wrote:
> > > One more behavior to the refresh-option below:
> > >
> > > If I just try to deactivate a Job on confirming I get the error-message:
> > > Cannot refresh browser
> > > There are properties dialogues open for one or more objects that would
> > > be refreshed. Please close the properties dialogues and try again.
> > >
> > > But I don't have any other dialogues open.
> > >
> >
> > I am not able to reproduce this case at my end with the steps as
> > given by you. Can you please provide me more information and
> > specific steps that you did to reproduce this error?
> >
> >  
> > > Regards
> > > Günther
> > >
> > >
> > >
> > >
> > >
> > > Hi all,
> > >
> > > Since I am using PgAdmin v.1.16.0 on Windows 7 64 bit when I
> change anything
> > > on a pgAgent job
> > > pgAgent chrashes.
> > > This happens only if I the new tree refresh option is turned on in
> > > Options->UI Miscellaneous->Refresh on click->Refresh object on click
> > > When I turn this option off it works fine.
> > >
> >
> > This crash was reproducible to me and am looking into it.
> >  
> > > Thank you!
> > >
> > > Günther
> > >
> > > --------------------------------------------------------------------------
> > > This email and any files transmitted with it are confidential and intended
> > > solely for the use of the individual or entity to whom they are addressed.
> > > If you have received this email in error please notify the system manager.
> > >
> > > Think GREEN before you print!
> > > Before you print, please consider if it is necessary.
> > > Printing uses electricity, ink and paper.
> > > --------------------------------------------------------------------------
> >
> >
>
> > --
> > Dave Page
> > Blog: http://pgsnake.blogspot.com
> > Twitter: @pgsnake
> >
> > EnterpriseDB UK: http://www.enterprisedb.com
> > The Enterprise PostgreSQL Company
> >
>
> >
> > --
> > regards,
> >
> > Dhiraj Chawla
> > Software Engineer
> > EnterpriseDB Corporation
> > The Enterprise PostgreSQL Company
> >
> > Phone: +91-20-30589522
> --------------------------------------------------------------------------
> This email and any files transmitted with it are confidential and intended
> solely for the use of the individual or entity to whom they are addressed.
> If you have received this email in error please notify the system manager.
>
> Think GREEN before you print!
> Before you print, please consider if it is necessary.
> Printing uses electricity, ink and paper.
> --------------------------------------------------------------------------
>

>
> --
> regards,
>
> Dhiraj Chawla
> Software Engineer
> EnterpriseDB Corporation
> The Enterprise PostgreSQL Company
>
> Phone: +91-20-30589522
--------------------------------------------------------------------------
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error please notify the system manager.

Think GREEN before you print!
Before you print, please consider if it is necessary.
Printing uses electricity, ink and paper.
--------------------------------------------------------------------------