Asim R P

This quick change seems to fix the issue but it could introduce new regressions. I don't want to check that now, let me note the change here instead. ```diff ---...

> (Why the COPY code needs to actually decompress a dropped column is left for another exercise, but it seems like the schema should match what's on disk in any...

The [6X documentation](https://docs.greenplum.org/6-10/admin_guide/ddl/ddl-table.html#topic34) refers to "randomly" and "round-robin fashion" interchangeably. My opinion is `distributed randomly` means no assumptions can be made about underlying data; motion node shouldn't be required when...

Do we have the same problem in `setupUDPListeningSocket()`? By the way, what exactly is the problem here? Is it a transient problem, does the error stop occurring after some time?

I see, thank you for the details. UDP connections won't suffer from this issue, `setupUDPListeningSocket()` doesn't need any fix. Am I right?

I agree with @paul-guo-, the fsync GUC has no effect on the fsync operation performed by `pg_basebackup` at the end of extracting the tar file at the destination. One code...

@ashwinstar I've changed the patch so that create-tablespace WAL record is not modified. The segment-specific subdir name continues to be a randomly generated string but we now create a symlink...

@kalensk raised a question about how this change impacts upgrade. The intention is that this change should not impact upgrade. Let’s consider a 6 to 7 upgrade scenario where user...

@gfphoenix78 this change eliminates the need to handle Greenplum-specific version within `pg_auto_failover` contrib module.

PR pipeline is red because of an unrelated test failure: ```diff --- /tmp/build/b235ddbb/gpdb_src/src/test/fsync/expected/bgwriter_checkpoint.out 2021-03-31 11:44:09.129029432 +0000 +++ /tmp/build/b235ddbb/gpdb_src/src/test/fsync/results/bgwriter_checkpoint.out 2021-03-31 11:44:09.133029776 +0000 @@ -202,7 +202,15 @@ as (tablespace oid, database oid,...