postgresql-embedded icon indicating copy to clipboard operation
postgresql-embedded copied to clipboard

pid fetch changed

Open miwroblewski opened this issue 5 years ago • 3 comments

142: Cached server path fails to find PID on Unix

Task-Url: https://github.com/yandex-qatools/postgresql-embedded/issues/142

miwroblewski avatar Sep 25 '18 19:09 miwroblewski

Hey @miwroblewski thank you for your contribution and sorry for the very late reply - I haven't been looking after the project for a while. Honestly, the change is looking good to me. I can't remember why there was so many dances with the pid file etc. Are you using it for your projects, does it work fine with just setting Pid from the Process?

smecsia avatar Nov 15 '18 10:11 smecsia

Hi, It worked pretty well, but I am not using the project anymore as I did face some issues with my Integration tests. Worth to mention I was testing it on my Linux Fedora 28, so not sure about other platforms (like Windows) so maybe it will require further testing for Windows.

miwroblewski avatar Nov 15 '18 11:11 miwroblewski

@miwroblewski, @smecsia - I strongly suggest to rethink this. As far as I understand, the complex code here is because the 'postgres.exe' is not the process which is executed. Instead pg_ctl or the 'runas' on Windows is executed, and they executed the postgres.exe. This causes problems during shutdown. @acommuni This is explained in https://github.com/yandex-qatools/postgresql-embedded/issues/129 in great detail. There's something odd in flapdoodle in how is uses the pid.

Doing this change here will make things even worse.

dbardbar avatar Dec 20 '18 07:12 dbardbar