Marco Slot

Would be good to add the coordinator to the metadata automatically. This is required for a lot of features now. ``` select create_distributed_table_concurrently('test','x'); ERROR: coordinator is not added to the...

I was confused why search_path would change without doing CREATE INDEX CONCURRENTLY Upon closer inspection, we should really be sending SET LOCAL search_path in this code path in utility_hook.c, which...

High-level thought: Should we use the clock value as part of the transaction ID timestamp instead of GetCurrentTimestamp? (probably guard this behaviour with a GUC in case it leads to...

Is there a design document for this PR? (ideally in a large comment or .md file, if we want to merge this) It seems interesting, but the guarantees we get,...

Btw, `make check` (in the root Makefile) depends on `make install` which removes the downgrade scripts. That's rather confusing.

> Do you think we can get the disk size per node so that we can display disk usage as you were also thinking? On the one hand, not in...

> I've updated naming scheme by including range values within partition name and truncating table name if necessary. I'll update the tests by thinking about timezones more. Just wondering, would...

Another error I'm hitting (using new binaries): ```sql postgres=# alter extension citus update TO "10.2-2"; ERROR:  cannot drop constraint stripe_first_row_number_idx on table columnar.stripe because other objects depend on it DETAIL:  ...

I'm kind of wondering whether we should try to preserve the shell table in undistribute/alter operations, to be able to keep the materialized views.

This behaviour is a bit strange: ``` $ cat /etc/security/limits.conf ... marco soft nice 0 marco hard nice 0 ``` ```sql $ psql begin; set local citus.cpu_priority = 1; select...