Commitebcc7bf

committed

Rephrase references to "time qualification".

Now that the relevant code has, for other reasons, moved out oftqual.[ch], it seems time to refer to visiblity rather than timequalification.Author: Andres FreundDiscussion:https://postgr.es/m/20180703070645.wchpu5muyto5n647@alap3.anarazel.de

1 parentc91560d commitebcc7bfCopy full SHA for ebcc7bf

File tree

-7

lines changed

-7

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -720,7 +720,7 @@ amparallelrescan (IndexScanDesc scan);`
`720`	`720`	`the TIDs of all the tuples it has been told about that match the`
`721`	`721`	`<firstterm>scan keys</firstterm>. The access method is <emphasis>not</emphasis> involved in`
`722`	`722`	`actually fetching those tuples from the index's parent table, nor in`
`723`		`- determining whether they pass the scan'stime qualification test or other`
	`723`	`+ determining whether they pass the scan'svisibility test or other`
`724`	`724`	`conditions.`
`725`	`725`	`</para>`
`726`	`726`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -1700,7 +1700,7 @@ heap_fetch(Relation relation,`
`1700`	`1700`	`tuple->t_tableOid=RelationGetRelid(relation);`
`1701`	`1701`
`1702`	`1702`	`/*`
`1703`		`- * checktime qualification oftuple, then release lock`
	`1703`	`+ * check tuple visibility, then release lock`
`1704`	`1704`	`*/`
`1705`	`1705`	`valid=HeapTupleSatisfiesVisibility(tuple,snapshot,buffer);`
`1706`	`1706`
`@@ -2020,8 +2020,8 @@ heap_get_latest_tid(Relation relation,`
`2020`	`2020`	`}`
`2021`	`2021`
`2022`	`2022`	`/*`
`2023`		`- * Checktime qualification oftuple; if visible, set it as the new`
`2024`		`- *resultcandidate.`
	`2023`	`+ * Check tuple visibility; if visible, set it as the new result`
	`2024`	`+ * candidate.`
`2025`	`2025`	`*/`
`2026`	`2026`	`valid=HeapTupleSatisfiesVisibility(&tp,snapshot,buffer);`
`2027`	`2027`	`CheckForSerializableConflictOut(valid,relation,&tp,buffer,snapshot);`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1453,8 +1453,8 @@ HeapTupleIsSurelyDead(HeapTuple htup, TransactionId OldestXmin)`
`1453`	`1453`	`* It's easy to check just infomask bits if the locker is not a multi; but`
`1454`	`1454`	`* otherwise we need to verify that the updating transaction has not aborted.`
`1455`	`1455`	`*`
`1456`		`- * This function is here because it follows the sametime qualificationrules`
`1457`		`- *laid outat the top of this file.`
	`1456`	`+ * This function is here because it follows the samevisibilityrules laid out`
	`1457`	`+ * at the top of this file.`
`1458`	`1458`	`*/`
`1459`	`1459`	`bool`
`1460`	`1460`	`HeapTupleHeaderIsOnlyLocked(HeapTupleHeadertuple)`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`*`
`6`	`6`	`*This is subtle stuff, so pay attention:`
`7`	`7`	`*`
`8`		`- *When a tuple is updated or deleted, our standardtime qualification rules`
	`8`	`+ *When a tuple is updated or deleted, our standardvisibility rules`
`9`	`9`	`consider that it is still valid* so long as we are in the same command,`
`10`	`10`	`*ie, until the next CommandCounterIncrement() or transaction commit.`
`11`	`11`	`*(See acces/heap/heapam_visibility.c, and note that system catalogs are`

Comments

(0)