Commit69c4319

danielgustafsson

authored and

pull[bot]

committed

Rephrase comments to make them clearer

The use of "we" when referring to the active backend might bemisunderstood, so rephrase to make it clearer who is performingthe actions discussed in the comment.Author: Junwang Zhao <zhjwpku@gmail.com>Reviewed-by: Erikjan Rijkers <er@xs4all.nl>Reviewed-by: Robert Treat <rob@xzilla.net>Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>Discussion:https://postgr.es/m/CAEG8a3LRSMqkvjiURiJoSi4aGWORpiXUmUfQQK5PaD6WfPzu3w@mail.gmail.com

1 parent2ddb6f6 commit69c4319Copy full SHA for 69c4319

File tree

1 file changed

-3

lines changed

src/backend/access/transam
- xact.c

1 file changed

-3

lines changed

`‎src/backend/access/transam/xact.c`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -3140,7 +3140,7 @@ CommitTransactionCommand(void)`
`3140`	`3140`	`break;`
`3141`	`3141`
`3142`	`3142`	`/*`
`3143`		`- *We were just issued a SAVEPOINT inside a transaction block.`
	`3143`	`+ *The user issued a SAVEPOINT inside a transaction block.`
`3144`	`3144`	`* Start a subtransaction. (DefineSavepoint already did`
`3145`	`3145`	`* PushTransaction, so as to have someplace to put the SUBBEGIN`
`3146`	`3146`	`* state.)`
`@@ -3151,7 +3151,7 @@ CommitTransactionCommand(void)`
`3151`	`3151`	`break;`
`3152`	`3152`
`3153`	`3153`	`/*`
`3154`		`- *We were issued a RELEASE command, so we end the current`
	`3154`	`+ *The user issued a RELEASE command, so we end the current`
`3155`	`3155`	`* subtransaction and return to the parent transaction. The parent`
`3156`	`3156`	`* might be ended too, so repeat till we find an INPROGRESS`
`3157`	`3157`	`* transaction or subtransaction.`
`@@ -3168,7 +3168,7 @@ CommitTransactionCommand(void)`
`3168`	`3168`	`break;`
`3169`	`3169`
`3170`	`3170`	`/*`
`3171`		`- *We were issued a COMMIT, so we end the current subtransaction`
	`3171`	`+ *The user issued a COMMIT, so we end the current subtransaction`
`3172`	`3172`	`* hierarchy and perform final commit. We do this by rolling up`
`3173`	`3173`	`* any subtransactions into their parent, which leads to O(N^2)`
`3174`	`3174`	`* operations with respect to resource owners - this isn't that`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit69c4319

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/xact.c`

0 commit comments