forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit6f4bdf8

Amit Kapila
Fix assertion during streaming of multi-insert toast changes.
While decoding the multi-insert WAL we can't clean the toast untill we getthe last insert of that WAL record. Now if we stream the changes before weget the last change, the memory for toast chunks won't be released and weexpect the txn to have streamed all changes after streaming. Thisrestriction is mainly to ensure the correctness of streamed transactionsand it doesn't seem worth uplifting such a restriction just to allow thiscase because anyway we will stream the transaction once such an insert iscomplete.Previously we were using two different flags (one for toast tuples andanother for speculative inserts) to indicate partial changes. Now insteadwe replaced both of them with a single flag to indicate partial changes.Reported-by: Pavan DeolaseeAuthor: Dilip KumarReviewed-by: Pavan Deolasee, Amit KapilaDiscussion:https://postgr.es/m/CABOikdN-_858zojYN-2tNcHiVTw-nhxPwoQS4quExeweQfG1Ug@mail.gmail.com1 parent190fa5a commit6f4bdf8
File tree
4 files changed
+73
-42
lines changed- contrib/test_decoding
- expected
- sql
- src
- backend/replication/logical
- include/replication
4 files changed
+73
-42
lines changedOriginal file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
82 | 82 |
| |
83 | 83 |
| |
84 | 84 |
| |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
85 | 109 |
| |
86 | 110 |
| |
87 | 111 |
| |
|
0 commit comments
Comments
(0)