Fix for RM2811

From: Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Fix for RM2811
Date: 2017-11-27 11:23:42
Message-ID: CAFiP3vycAMc3WJQm7ZBkRg4vMy32oy3jp6Sd+yQTu3sNB0Sv3Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

Please find attached patch to fix RM2811

Issue was caused due to assumption made when current position in log file
while reading it reaches to last line then
we were assuming process is finished. However this is not the case.
Background process may be busy performing
some other task and logs might not be logged to file immediately. So we
should also check process exit code along
with above condition.

Apart from above this patch also includes minor fix related to status text
colour.

--
*Harshal Dhumal*
*Sr. Software Engineer*

EnterpriseDB India: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment Content-Type Size
RM2811.patch text/x-patch 3.2 KB

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-11-27 11:25:54 pgAdmin 4 commit: Ensure we don't show the full path on the server when
Previous Message Dave Page 2017-11-27 11:17:13 Re: [pgAdmin4][Patch]: RM-2859: Can't create new user