Обсуждение: [pgAdmin4][Patch]: Stop Flask from initialising service twice inDebug mode

Поиск
Список
Период
Сортировка

[pgAdmin4][Patch]: Stop Flask from initialising service twice inDebug mode

От
Surinder Kumar
Дата:
Hi

When pgAdmin4 runs in DEBUG mode, Flask initialise service twice, while in non-DEBUG mode, once. It is pain while development because `yarn run bundle` also runs twice.

It is because the parameter `use_reloader` = True in Debug Mode.

If `os.environ` variable `WERKZEUG_RUN_MAIN` is set to None, i.e. app is starting for the first time and then its value is changed to True.

Please find attached patch.

Thanks
Surinder



Вложения

Re: [pgAdmin4][Patch]: Stop Flask from initialising service twice inDebug mode

От
Surinder Kumar
Дата:
Hi

In this patch, i have added reference to the implementation.
Please find updated patch and review.

Thanks,
Surinder

On Fri, Jul 21, 2017 at 12:32 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

When pgAdmin4 runs in DEBUG mode, Flask initialise service twice, while in non-DEBUG mode, once. It is pain while development because `yarn run bundle` also runs twice.

It is because the parameter `use_reloader` = True in Debug Mode.

If `os.environ` variable `WERKZEUG_RUN_MAIN` is set to None, i.e. app is starting for the first time and then its value is changed to True.

Please find attached patch.

Thanks
Surinder




Вложения

Re: [pgAdmin4][Patch]: Stop Flask from initialising service twice inDebug mode

От
Dave Page
Дата:
Thanks, applied. The reloader has always been a little temperamental for me, so I couldn't prove this doesn't break it. If someone finds it does, please yelp. 

On Fri, Jul 21, 2017 at 8:27 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

In this patch, i have added reference to the implementation.
Please find updated patch and review.

Thanks,
Surinder

On Fri, Jul 21, 2017 at 12:32 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

When pgAdmin4 runs in DEBUG mode, Flask initialise service twice, while in non-DEBUG mode, once. It is pain while development because `yarn run bundle` also runs twice.

It is because the parameter `use_reloader` = True in Debug Mode.

If `os.environ` variable `WERKZEUG_RUN_MAIN` is set to None, i.e. app is starting for the first time and then its value is changed to True.

Please find attached patch.

Thanks
Surinder







--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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

Re: [pgAdmin4][Patch]: Stop Flask from initialising service twice inDebug mode

От
Matthew Kleiman
Дата:
Surinder, you are my hero! 🎁

On Fri, Jul 21, 2017 at 4:47 AM, Dave Page <dpage@pgadmin.org> wrote:
Thanks, applied. The reloader has always been a little temperamental for me, so I couldn't prove this doesn't break it. If someone finds it does, please yelp. 

On Fri, Jul 21, 2017 at 8:27 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

In this patch, i have added reference to the implementation.
Please find updated patch and review.

Thanks,
Surinder

On Fri, Jul 21, 2017 at 12:32 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi

When pgAdmin4 runs in DEBUG mode, Flask initialise service twice, while in non-DEBUG mode, once. It is pain while development because `yarn run bundle` also runs twice.

It is because the parameter `use_reloader` = True in Debug Mode.

If `os.environ` variable `WERKZEUG_RUN_MAIN` is set to None, i.e. app is starting for the first time and then its value is changed to True.

Please find attached patch.

Thanks
Surinder







--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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