Bump PG versions to 17.6, 16.10, 15.14

[naisila]

Sister PR citusdata/the-process#172

Fixes #8134 #8149

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.93%. Comparing base (aaa3137) to head (c6c80de).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8142      +/-   ##
==========================================
+ Coverage   88.90%   88.93%   +0.03%     
==========================================
  Files         287      287              
  Lines       63083    63088       +5     
  Branches     7930     7930              
==========================================
+ Hits        56083    56107      +24     
+ Misses       4693     4673      -20     
- Partials     2307     2308       +1     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[naisila]

Ran local tests with PG15 & PG17. No more assertion crashes.

[m3hm3t]

nitpick:

we may reduce code duplication by similar like

/* maybe under a utils file */
static inline void
CatalogInsertWithTxnSnapshot(Relation rel, HeapTuple tup)
{
    PushActiveSnapshot(GetTransactionSnapshot());
    CatalogTupleInsert(rel, tup);
    PopActiveSnapshot();
}

or more complex but more generic and can be used later as well

#define WITH_TXN_SNAPSHOT(...)                        \
    do {                                              \
        PushActiveSnapshot(GetTransactionSnapshot()); \
        __VA_ARGS__;                                  \
        PopActiveSnapshot();                          \
    } while (0)

WITH_TXN_SNAPSHOT(
    InsertTupleAndEnforceConstraints(modifyState, values, nulls)
);

WITH_TXN_SNAPSHOT(
    CatalogTupleInsert(pgDistTransaction, heapTuple)
);

[naisila]

Prefer to go with Postgres approach postgres/postgres@706054b#diff-d373fb790605aded212c2e3a0fbe8c015d110077ff46c07d0a6d6647a2d7e928

[m3hm3t]

nothing other than a nitpick, thanks

[ivan-v-kush]

@naisila could you please tell why PushActiveSnapshot/PopActiveSnapshot are inside the loop rather than outside
Is it possible to do it this way to save resources?

	PushActiveSnapshot(GetTransactionSnapshot());
	for (columnIndex = 0; columnIndex < columnCount; columnIndex++)
	{
		for (chunkIndex = 0; chunkIndex < chunkList->chunkCount; chunkIndex++)
		{
			....
			InsertTupleAndEnforceConstraints(modifyState, values, nulls);
			...
		}
		...
	}
	PopActiveSnapshot();

your code:

	for (columnIndex = 0; columnIndex < columnCount; columnIndex++)
	{
		for (chunkIndex = 0; chunkIndex < chunkList->chunkCount; chunkIndex++)
		{
			...
			PushActiveSnapshot(GetTransactionSnapshot());

			InsertTupleAndEnforceConstraints(modifyState, values, nulls);

			PopActiveSnapshot();
			...
		}
		...
	}

Thank you!