public inbox for [email protected]
help / color / mirror / Atom feedFrom: Neel Patel <[email protected]>
To: Dave Page <[email protected]>
Cc: Murtuza Zabuawala <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: PATCH: To fix the issue in Debugger module (pgAdmin4)
Date: Fri, 21 Oct 2016 17:02:37 +0530
Message-ID: <CACCA4P2BOCiNDisEfTi_aSTSugUFTjZea-YZ-AWZNSBN8Lj63w@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxoyCRkmA29pCZh9PhG0ZSLgRYwX+5yyqHCYTxZjkDVHcHw@mail.gmail.com>
References: <CAKKotZQz6mGEiky_Szo91tzm_RFvwJTYGriTmuxVYrhcWi5nJg@mail.gmail.com>
<CA+OCxoxkphr1YF6F53j3sTreTMvoBOKqV+wjF6CnxoQ-SA+rzA@mail.gmail.com>
<CAKKotZRMetEEPzFTuqiw_8Vit2_STHSN22=K+5frJBu7=Jp6PQ@mail.gmail.com>
<CAKKotZQGdhiFrtNK0xz8OVLKOA2QCk_yfOZDFT+HEMfELzbk4A@mail.gmail.com>
<CA+OCxoyzS4_cST0qJ7CbmxqOry4wGZCQkoiLpcXgjr2N4snPQw@mail.gmail.com>
<CAKKotZSjyeZT_bS8_No-cEftRjqPPptLx+zV2+_DqV=KYZYRcw@mail.gmail.com>
<CA+OCxoyLMvyRpPSMztrUf0st7yGvyeYquOmyFabQzg6Czieh=Q@mail.gmail.com>
<CAKKotZRW4WF2gsaPu-cRZJmrAia0GiDTOhwDKQdUpBC5VRoVjQ@mail.gmail.com>
<CA+OCxoyF37xc+S4=r37KRAEh+zUfN4-WBSSv1YSe1VZpMmNfkg@mail.gmail.com>
<CAKKotZSz3kgLYGNyu_BGORF+J=F2qmLa-2z9Rzq1HgSjXjTyZg@mail.gmail.com>
<CA+OCxoymEzDsLaABZixOjQ37jJdP1J7+xJ6By_oJHi=2bSRoRg@mail.gmail.com>
<CAKKotZSrWzCVBbuywF8VFGMtMK-VdT=fyo_3gY97OnoO7hZspw@mail.gmail.com>
<CA+OCxowWbW6sYELhWiCGW3znKE6Fk19Hwuns8BzZHd94iqABnA@mail.gmail.com>
<CA+OCxozg9xXuCy86gaxvnfX646XpeP5c9g0ycW1oE-k-jg22cA@mail.gmail.com>
<CAKKotZRA0sLv6Aiua2h8FXpyqrjXg=Wt6WipVK4O_xCpdnfvgA@mail.gmail.com>
<CA+OCxoyCRkmA29pCZh9PhG0ZSLgRYwX+5yyqHCYTxZjkDVHcHw@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
Hi,
On Fri, Oct 21, 2016 at 4:48 PM, Dave Page <[email protected]> wrote:
> Hi
>
> There are still issues I'm afraid:
>
> - When execution stops, we seem to keep polling for more results
> indefinitely.
>
Do you mean after completion of first successful debugging ?
If yes, we are polling because user can start same function for debugging
again and we have to listen for the result set for that session.
>
> - When executing for a second time, the messages tab isn't cleared,
> and new messages don't seem to be appended to it either. I would
> expect the tab to be cleared.
>
Ok. We will fix this issue.
>
> On Thu, Oct 20, 2016 at 9:14 AM, Murtuza Zabuawala
> <[email protected]> wrote:
> > Hi Dave,
> >
> > PFA updated patch for the same.
> >
> > Issue:
> > We were not properly fetching result from server in case of direct
> debugging
> > when we restart debugging of same object.
> >
> > Thanks to Neel for helping in this issue.
> >
> > Please review.
> >
> > --
> > Regards,
> > Murtuza Zabuawala
> > EnterpriseDB: http://www.enterprisedb.com
> > The Enterprise PostgreSQL Company
> >
> > On Fri, Oct 7, 2016 at 5:32 PM, Dave Page <[email protected]> wrote:
> >>
> >> On Fri, Oct 7, 2016 at 12:53 PM, Dave Page <[email protected]> wrote:
> >> > On Fri, Oct 7, 2016 at 12:42 PM, Murtuza Zabuawala
> >> > <[email protected]> wrote:
> >> >> Hi Dave,
> >> >>
> >> >> I faced the same issue when I initially tried that, but then as per
> >> >> Neel
> >> >> suggestion I changed SELECT pg_sleep() to PERFORM pg_sleep() in
> >> >> function.
> >> >> You will face the same in pgAdmin3 if you use select pg_sleep() in
> your
> >> >> function the debug call never returns from DB server.
> >> >
> >> > In which case, doesn't that imply the debugger is missing critical
> >> > debug info? If I run the query in the query tool, I get:
> >> >
> >> > ====
> >> > INFO: EMPNO ENAME
> >> > INFO: ----- -------
> >> > ERROR: query has no destination for result data
> >> > HINT: If you want to discard the results of a SELECT, use PERFORM
> >> > instead.
> >> > CONTEXT: PL/pgSQL function list_emp() line 11 at SQL statement
> >> >
> >> >
> >> > Query returned successfully in 2 secs.
> >> > ====
> >> >
> >> > It seems to me that the debugger should be able to give the same
> error.
> >> >
> >> > Regardless of that, I'll test with PERFORM.
> >>
> >> Which I just did - and whilst it seemed to be fine when stepping
> >> through, after a few iterations I hit the continue button, at which
> >> point it froze again on "PERFORM pg_sleep(2)", didn't print any more
> >> of the 14 names in the emp table, and didn't return :-(
> >>
> >> --
> >> Dave Page
> >> Blog: http://pgsnake.blogspot.com
> >> Twitter: @pgsnake
> >>
> >> EnterpriseDB UK: http://www.enterprisedb.com
> >> The Enterprise PostgreSQL Company
> >
> >
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list ([email protected])
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
view thread (21+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: PATCH: To fix the issue in Debugger module (pgAdmin4)
In-Reply-To: <CACCA4P2BOCiNDisEfTi_aSTSugUFTjZea-YZ-AWZNSBN8Lj63w@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox