8000 Fix docs regarding AFTER triggers on partitioned tables · postgres/postgres@3acfe6b · GitHub
[go: up one dir, main page]
Skip to content
{"payload":{"commit":{"oid":"3acfe6b089a5283c2961eb2ff135106e23b89263","url":"/postgres/postgres/commit/3acfe6b089a5283c2961eb2ff135106e23b89263","authoredDate":"2020-02-26T19:57:14.000-03:00","committedDate":"2020-02-26T19:57:14.000-03:00","shortMessage":null,"shortMessageMarkdown":"\u003cdiv\u003eFix docs regarding AFTER triggers on partitioned tables\u003c/div\u003e","shortMessageMarkdownLink":null,"bodyMessageHtml":"In commit \u003ca class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/postgres/postgres/commit/86f575948c773b0ec5b0f27066e37dd93a7f0a96/hovercard\" href=\"https://github.com/postgres/postgres/commit/86f575948c773b0ec5b0f27066e37dd93a7f0a96\"\u003e\u003ctt\u003e86f5759\u003c/tt\u003e\u003c/a\u003e I forgot to update the trigger.sgml paragraph\nthat needs to explain that AFTER triggers are allowed in partitioned\ntables. Do so now.\n\nDiscussion: \u003ca href=\"https://postgr.es/m/20200224185850.GA30899@alvherre.pgsql\" rel=\"nofollow\"\u003ehttps://postgr.es/m/20200224185850.GA30899@alvherre.pgsql\u003c/a\u003e","authors":[{"login":"alvherre","displayName":"Alvaro Herrera","avatarUrl":"https://avatars.githubusercontent.com/u/340005?v=4","path":"/alvherre","isGitHub":false}],"committerAttribution":false,"committer":{"login":"alvherre","displayName":"Alvaro Herrera","avatarUrl":"https://avatars.githubusercontent.com/u/340005?v=4","path":"/alvherre","isGitHub":false},"parents":["2d8a6fad18fcec1850b79ddcf3a0032058d86715"],"globalRelayId":"MDY6Q29tbWl0OTI3NDQyOjNhY2ZlNmIwODlhNTI4M2MyOTYxZWIyZmYxMzUxMDZlMjNiODkyNjM=","sha1":"2d8a6fad18fcec1850b79ddcf3a0032058d86715","sha2":"3acfe6b089a5283c2961eb2ff135106e23b89263"},"currentUser":null,"repo":{"id":927442,"defaultBranch":"master","name":"postgres","ownerLogin":"postgres","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2010-09-21T11:35:45.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/177543?v=4","public":true,"private":false,"isOrgOwned":true},"diffEntryData":[{"diffLines":[{"stylingDirective":null,"type":"HUNK","blobLineNumber":114,"text":"@@ -115,8 +115,10 @@","html":"@@ -115,8 +115,10 @@","displayNoNewLineWarning":false,"position":0,"left":114,"right":114},{"stylingDirective":null,"type":"CONTEXT","blobLineNumber":115,"text":" \u003cliteral\u003eBEFORE\u003c/literal\u003e triggers fire immediately before a particular row is","html":" \u0026lt;literal\u0026gt;BEFORE\u0026lt;/literal\u0026gt; triggers fire immediately before a particular row is","displayNoNewLineWarning":false,"position":1,"left":115,"right":115},{"stylingDirective":null,"type":"CONTEXT","blobLineNumber":116,"text":" operated on, while row-level \u003cliteral\u003eAFTER\u003c/literal\u003e triggers fire at the end of","html":" operated on, while row-level \u0026lt;literal\u0026gt;AFTER\u0026lt;/literal\u0026gt; triggers fire at the end of","displayNoNewLineWarning":false,"position":2,"left":116,"right":116},{"stylingDirective":null,"type":"CONTEXT","blobLineNumber":117,"text":" the statement (but before any statement-level \u003cliteral\u003eAFTER\u003c/literal\u003e triggers).","html":" the statement (but before any statement-level \u0026lt;literal\u0026gt;AFTER\u0026lt;/literal\u0026gt; triggers).","displayNoNewLineWarning":false,"position":3,"left":117,"right":117},{"stylingDirective":null,"type":"DELETION","blobLineNumber":118,"text":"- These types of triggers may only be defined on non-partitioned tables and","html":"- These types of triggers may only be defined on non-partitioned tables and","displayNoNewLineWarning":false,"position":4,"left":118,"right":117},{"stylingDirective":null,"type":"DELETION","blobLineNumber":119,"text":"- foreign tables, not views. \u003cliteral\u003eINSTEAD OF\u003c/literal\u003e triggers may only be","html":"- foreign tables, not views. \u0026lt;literal\u0026gt;INSTEAD OF\u0026lt;/literal\u0026gt; triggers may only be","displayNoNewLineWarning":false,"position":5,"left":119,"right":117},{"stylingDirective":null,"type":"ADDITION","blobLineNumber":118,"text":"+ These types of triggers may only be defined on tables and","html":"+ These types of triggers may only be defined on tables and","displayNoNewLineWarning":false,"position":6,"left":119,"right":118},{"stylingDirective":null,"type":"ADDITION","blobLineNumber":119,"text":"+ foreign tables, not views; \u003cliteral\u003eBEFORE\u003c/literal\u003e row-level triggers may not","html":"+ foreign tables, not views; \u0026lt;literal\u0026gt;BEFORE\u0026lt;/literal\u0026gt; row-level triggers may not","displayNoNewLineWarning":false,"position":7,"left":119,"right":119},{"stylingDirective":null,"type":"ADDITION","blobLineNumber":120,"text":"+ be defined on partitioned tables.","html":"+ be defined on partitioned tables.","displayNoNewLineWarning":false,"position":8,"left":119,"right":120},{"stylingDirective":null,"type":"ADDITION","blobLineNumber":121,"text":"+ \u003cliteral\u003eINSTEAD OF\u003c/literal\u003e triggers may only be","html":"+ \u0026lt;literal\u0026gt;INSTEAD OF\u0026lt;/literal\u0026gt; triggers may only be","displayNoNewLineWarning":false,"position":9,"left":119,"right":121},{"stylingDirective":null,"type":"CONTEXT","blobLineNumber":122,"text":" defined on views, and only at row level; they fire immediately as each","html":" defined on views, and only at row level; they fire immediately as each","displayNoNewLineWarning":false,"position":10,"left":120,"right":122},{"stylingDirective":null,"type":"CONTEXT","blobLineNumber":123,"text":" row in the view is identified as needing to be operated on.","html":" row in the view is identified as needing to be operated on.","displayNoNewLineWarning":false,"position":11,"left":121,"right":123},{"stylingDirective":null,"type":"CONTEXT","blobLineNumber":124,"text":" \u003c/para\u003e","html":" \u0026lt;/para\u0026gt;","displayNoNewLineWarning":false,"position":12,"left":122,"right":124}],"diffNumber":0,"diffSize":"0 Bytes","isBinary":false,"isTooBig":false,"collapsed":false,"isSubmodule":false,"lineCount":976,"linesChanged":6,"newTreeEntry":{"lineCount":976,"path":"doc/src/sgml/trigger.sgml","mode":100644,"isGenerated":false},"oldTreeEntry":{"lineCount":0,"path":"doc/src/sgml/trigger.sgml","mode":100644},"linesAdded":4,"linesDeleted":2,"path":"doc/src/sgml/trigger.sgml","pathDigest":"8637438928a7ff97d9ae2316375ff46c24ddf36e5e6a49ccd129d40b7d5db303","status":"MODIFIED","truncatedReason":null,"oldOid":"2d8a6fad18fcec1850b79ddcf3a0032058d86715","newOid":"3acfe6b089a5283c2961eb2ff135106e23b89263","copilotChatReference":null,"deletedSha":"2d8a6fad18fcec1850b79ddcf3a0032058d86715","canToggleRichDiff":false,"defaultToRichDiff":false,"proseDifffHtml":null,"renderInfo":null,"dependencyDiffPath":null,"submodule":null}],"splitViewPreference":"unified","ignoreWhitespace":false,"repoOwnerGlobalRelayId":"MDEyOk9yZ2FuaXphdGlvbjE3NzU0Mw==","commentsPreference":"visible","diffLineSpacingPreference":"relaxed","useMonospaceFont":false,"pasteUrlLinkAsPlainText":false,"userNotices":[],"path":"/postgres/postgres/commit/3acfe6b089a5283c2961eb2ff135106e23b89263","fileTreeExpanded":true,"headerInfo":{"additions":4,"deletions":2,"filesChanged":1,"filesChangedString":"1"},"moreDiffsToLoad":false,"asyncDiffLoadInfo":{"startIndex":1,"truncated":false,"byteCount":837,"lineShownCount":13},"commentInfo":{"canComment":false,"locked":false,"canLock":false,"repoArchived":false},"csrf_tokens":{"/users/diffview?diff=split":{"post":"sxRkcVMuN0Iemi5uX-IARvHbPiEA1GH-VX-AxqdcSGcPdLFsnYecmrnOyhunsmNpwslTmtyQ4VyLU22l80YEng"},"/users/diffview?diff=unified":{"post":"Qw7byBAxf-DCJ-i2HUTl0KrPD7BqR3HRE087FNCWSBD_bg7V3pjUOGVzDMPlFIb_md1iC7YD8XPNY9Z3hIwE6Q"},"/notifications/thread":{"post":"rGqx0Sr1r7RX2WUk2BVv1akAZolc-c3gRTJJooZyqw4U_-BxB30g_mQTL8C5tnwUfkfobjotW8vJBWgDgehObQ"}}},"title":"Fix docs regarding AFTER triggers on partitioned tables · postgres/postgres@3acfe6b","appPayload":{"helpUrl":"https://docs.github.com","findInDiffWorkerPath":"/assets-cdn/worker/find-in-diff-worker-2bfe39677d14.js","enabled_features":{"diff_ux_refresh_beta":false,"diff_inline_comments":true,"diff_ux_refresh_ssr_five":false,"diff_ux_refresh_ssr_ten":false,"react_diff_line_type_character_correction":true}}}

Commit 3acfe6b

Browse files
alvherrealvherre
committed
Fix docs regarding AFTER triggers on partitioned tables
In commit 86f5759 I forgot to update the trigger.sgml paragraph that needs to explain that AFTER triggers are allowed in partitioned tables. Do so now. Discussion: https://postgr.es/m/20200224185850.GA30899@alvherre.pgsql
1 parent 2d8a6fa commit 3acfe6b

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

‎doc/src/sgml/trigger.sgml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,8 +115,10 @@
115115
<literal>BEFORE</literal> triggers fire immediately before a particular row is
116116
operated on, while row-level <literal>AFTER</literal> triggers fire at the end of
117117
the statement (but before any statement-level <literal>AFTER</literal> triggers).
118-
These types of triggers may only be defined on non-partitioned tables and
119-
foreign tables, not views. <literal>INSTEAD OF</literal> triggers may only be
118+
These types of triggers may only be defined on tables and
119+
foreign tables, not views; <literal>BEFORE</literal> row-level triggers may not
120+
be defined on partitioned tables.
121+
<literal>INSTEAD OF</literal> triggers may only be
120122
defined on views, and only at row level; they fire immediately as each
121123
row in the view is identified as needing to be operated on.
122124
</para>

0 commit comments

Comments
 (0)
0