Re: Fix for RM2811
От | Harshal Dhumal |
---|---|
Тема | Re: Fix for RM2811 |
Дата | |
Msg-id | CAFiP3vxQFa7eHfe-r1nwLU76x01ys=txGmTxAGs2aCBeeXP7nw@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Fix for RM2811 (Dave Page <dpage@pgadmin.org>) |
Ответы |
Re: Fix for RM2811
|
Список | pgadmin-hackers |
Hi Dave,
Please find updated patch.
--
Harshal Dhumal
Sr. Software Engineer
On Mon, Nov 27, 2017 at 5:28 PM, Dave Page <dpage@pgadmin.org> wrote:
HiOn Mon, Nov 27, 2017 at 11:23 AM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote: Hi,Please find attached patch to fix RM2811Issue was caused due to assumption made when current position in log file while reading it reaches to last line thenwe were assuming process is finished. However this is not the case. Background process may be busy performingsome other task and logs might not be logged to file immediately. So we should also check process exit code alongwith above condition.Apart from above this patch also includes minor fix related to status text colour.Seems to work nicely, except that the text scrolls out of view almost immediately. Can you fix it to jump to the end of the text when more is appended please?
Fixed.
Thanks.--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Вложения
В списке pgadmin-hackers по дате отправления: