Ashwin Agrawal

@BaiShaoqi this change seems good to pursue, please lets try to get this to finish line seems are are almost there.

PR has been sitting open for more than 2 years, seems no interest / need to drive it to completion. Hence, closing the same.

- Always helpful practice to follow to have code indentation fixes to be in separate commit from enhancements/fixes - This enhancement of functionality of infra "Added get function to worker...

Please can you high-level explain why gprecoverseg goes at this length to check if postmaster is running for the given data directory by parsing the PID file and all such...

> gprecoverseg does all the above checks on PID before going to stop the segment to make sure that we are stopping the correct segment and the PID mentioned is...

> > > gprecoverseg does all the above checks on PID before going to stop the segment to make sure that we are stopping the correct segment and the PID...

> > > Question here is, if we remove that part, during rebalance operation we try to stop non existent process and print following message: > > > > >...

@kaknikhil @jmcatamney please can you help review this one-line PR from our python code perspective

@vraghavan78 please assess if wish to fix this for 6X_STABLE if not please feel free to close this issue as doesn't exist for gpdb7 and forward

Merged via commit 0d4153fb0ee62db2cab30ff3cc79ae574a5d66c0