forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit095d109
committed
Remove redundant setting of hashkey after insertion
It's not necessary to fill the key field in most cases, sincehash_search has already done that. Some existing call sites have anassert or comment that this contract has been fulfilled, but thoseare quite old and that practice seems unnecessary here.While at it, remove a nearby redundant assignment that a smart compilerwill elide anyway.Zhao Junwang, with some adjustments by meReviewed by Nathan Bossart, with additional feedback from Tom LaneDiscussion:http://postgr.es/m/CAEG8a3%2BUPF%3DR2QGPgJMF2mKh8xPd1H2TmfH77zPuVUFdBpiGUA%40mail.gmail.com1 parent489ca33 commit095d109
File tree
5 files changed
+8
-18
lines changed- contrib/dblink
- src/backend
- commands
- replication/logical
5 files changed
+8
-18
lines changedLines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2574 | 2574 |
| |
2575 | 2575 |
| |
2576 | 2576 |
| |
2577 |
| - | |
2578 | 2577 |
| |
2579 | 2578 |
| |
2580 | 2579 |
| |
|
Lines changed: 8 additions & 12 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2305 | 2305 |
| |
2306 | 2306 |
| |
2307 | 2307 |
| |
2308 |
| - | |
2309 | 2308 |
| |
2310 | 2309 |
| |
2311 |
| - | |
2312 |
| - | |
2313 |
| - | |
2314 |
| - | |
| 2310 | + | |
| 2311 | + | |
| 2312 | + | |
| 2313 | + | |
2315 | 2314 |
| |
2316 |
| - | |
2317 | 2315 |
| |
2318 | 2316 |
| |
2319 | 2317 |
| |
| |||
2323 | 2321 |
| |
2324 | 2322 |
| |
2325 | 2323 |
| |
2326 |
| - | |
2327 | 2324 |
| |
2328 | 2325 |
| |
2329 |
| - | |
2330 |
| - | |
2331 |
| - | |
2332 |
| - | |
| 2326 | + | |
| 2327 | + | |
| 2328 | + | |
| 2329 | + | |
2333 | 2330 |
| |
2334 |
| - | |
2335 | 2331 |
| |
2336 | 2332 |
| |
2337 | 2333 |
| |
|
Lines changed: 0 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2104 | 2104 |
| |
2105 | 2105 |
| |
2106 | 2106 |
| |
2107 |
| - | |
2108 |
| - | |
2109 | 2107 |
| |
2110 |
| - | |
2111 | 2108 |
| |
2112 | 2109 |
| |
2113 | 2110 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
509 | 509 |
| |
510 | 510 |
| |
511 | 511 |
| |
512 |
| - | |
513 | 512 |
| |
514 | 513 |
| |
515 | 514 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
657 | 657 |
| |
658 | 658 |
| |
659 | 659 |
| |
660 |
| - | |
661 | 660 |
| |
662 | 661 |
| |
663 | 662 |
| |
|
0 commit comments
Comments
(0)