Fix comments in execTuples.c - Mailing list pgsql-hackers

From cca5507
Subject Fix comments in execTuples.c
Date
Msg-id tencent_89E1211F7CD96D842614A6D6F2A7BCCA440A@qq.com
Whole thread Raw
Responses Re: Fix comments in execTuples.c
List pgsql-hackers
Hi hackers,

```
diff --git a/src/backend/executor/execTuples.c b/src/backend/executor/execTuples.c
index 8e02d68824f..b77e9513e7e 100644
--- a/src/backend/executor/execTuples.c
+++ b/src/backend/executor/execTuples.c
@@ -1532,9 +1532,6 @@ ExecSetSlotDescriptor(TupleTableSlot *slot, /* slot to change */
  * and let the upper-level table slot assume ownership of the copy!
  *
  * Return value is just the passed-in slot pointer.
- *
- * If the target slot is not guaranteed to be TTSOpsHeapTuple type slot, use
- * the, more expensive, ExecForceStoreHeapTuple().
  * --------------------------------
  */
 TupleTableSlot *
@@ -1572,9 +1569,6 @@ ExecStoreHeapTuple(HeapTuple tuple,
  * slot is cleared, so that the tuple won't go away on us.
  *
  * Return value is just the passed-in slot pointer.
- *
- * If the target slot is not guaranteed to be TTSOpsBufferHeapTuple type slot,
- * use the, more expensive, ExecForceStoreHeapTuple().
  * --------------------------------
  */
 TupleTableSlot *
@@ -1627,9 +1621,6 @@ ExecStorePinnedBufferHeapTuple(HeapTuple tuple,

 /*
  * Store a minimal tuple into TTSOpsMinimalTuple type slot.
- *
- * If the target slot is not guaranteed to be TTSOpsMinimalTuple type slot,
- * use the, more expensive, ExecForceStoreMinimalTuple().
  */
 TupleTableSlot *
 ExecStoreMinimalTuple(MinimalTuple mtup,
```

All of these are reporting an error rather than using ExecForceStore*(), so I just remove these comments.

Thoughts?

--
Regards,
ChangAo Chen

Attachment

pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Proposal: Conflict log history table for Logical Replication
Next
From: Nazir Bilal Yavuz
Date:
Subject: Re: headerscheck ccache support