On Wed, Feb 28, 2018 at 11:35 PM, Thomas Munro
<[email protected]> wrote:
> On Mon, Feb 26, 2018 at 6:37 PM, Thomas Munro
> <[email protected]> wrote:
>> I've now broken it into two patches.
>
> Rebased.
+SerializableXactHandle
+ShareSerializableXact(void)
+{
+ Assert(!IsParallelWorker());
+
+ return MySerializableXact;
+}
Uh, how's that OK? There's no rule that you can't create a
ParallelContext in a worker. Parallel query currently doesn't, so it
probably won't happen, but burying an assertion to that effect in the
predicate locking code doesn't seem nice.
Is "sxact" really the best (i.e. clearest) name we can come up with
for the lock tranche?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
