Zhenglong Li

Not found the culprit yet, still in progress.

@InnerLife0 Sorry for the late response, after I changed the number of segment from 2 to 3 in test pipeline environment, the `resgroup_move_query` case still failed, failed at "test10: check...

> I believe the answer is not. > > The only concern is during greenplum's planning stage, we will evaluate stable function, and a stable function might query a distributed...

> Hi, thanks for your contribution. Following are my thoughts to involve more discussion. > > Consider a distributed transaction `insert into` to load data to each segment, say `seg0,...

> do not get distributed snapshot in the main code path > only fetch a new snapshot when SPI evaluate SQL for stable function. (rare cases) But if we do...

This PR has been silent for a long time, have any new questions or thoughts? @avamingli @kainwen @ashwinstar

I have updated the test cases, if somebody has time, take a look, thanks.

> Some comments: > > 1. can `update|delete|insert` benefited by this idea? > 2. can we make the function `checkNeedDistributedSnapshot` a `static bool` 1. No, because `update|delete|insert` will not use...

> > 1. No, because `update|delete|insert` will not use `PORTAL_ONE_SELECT` strategy. > > Can we take a look at which strategy they use and if we can use the same...

> Could someone please post the failed tests' output in concourse-ci? It failed at `gporca`, which will output the detail of type conversion: ``` ====================================================================== DIFF FILE: ../gpdb_src/src/test/regress/regression.diffs ---------------------------------------------------------------------- ---...