forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit22c5e73
committed
Remove lsn from HashScanPosData.
This was intended as infrastructure for weakening VACUUM's lockingrequirements, similar to what was done for btree indexes in commit2ed5b87. However, for hash indexes,it seems that the improvements which are possible are actuallyextremely marginal. Furthermore, performing the LSN cross-check willend up skipping cleanup far more often than is necessary; we only careabout page modifications due to a VACUUM, but the LSN check will failif ANY modification has occurred. So, rather than pressing forwardwith that "optimization", just rip the LSN field out.Patch by me, reviewed by Ashutosh Sharma and Amit KapilaDiscussion:http://postgr.es/m/CAA4eK1JxqqcuC5Un7YLQVhOYSZBS+t=3xqZuEkt5RyquyuxpwQ@mail.gmail.com1 parent79a4a66 commit22c5e73
File tree
3 files changed
+7
-30
lines changed- src
- backend/access/hash
- include/access
3 files changed
+7
-30
lines changedLines changed: 0 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
463 | 463 |
| |
464 | 464 |
| |
465 | 465 |
| |
466 |
| - | |
467 |
| - | |
468 |
| - | |
469 |
| - | |
470 |
| - | |
471 |
| - | |
472 | 466 |
| |
473 | 467 |
| |
474 | 468 |
| |
| |||
508 | 502 |
| |
509 | 503 |
| |
510 | 504 |
| |
511 |
| - | |
512 | 505 |
| |
513 | 506 |
| |
514 | 507 |
| |
| |||
562 | 555 |
| |
563 | 556 |
| |
564 | 557 |
| |
565 |
| - | |
566 | 558 |
| |
567 | 559 |
| |
568 | 560 |
| |
|
Lines changed: 7 additions & 20 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
532 | 532 |
| |
533 | 533 |
| |
534 | 534 |
| |
535 |
| - | |
536 |
| - | |
537 |
| - | |
538 |
| - | |
539 |
| - | |
540 |
| - | |
| 535 | + | |
| 536 | + | |
| 537 | + | |
| 538 | + | |
| 539 | + | |
| 540 | + | |
| 541 | + | |
541 | 542 |
| |
542 | 543 |
| |
543 | 544 |
| |
| |||
579 | 580 |
| |
580 | 581 |
| |
581 | 582 |
| |
582 |
| - | |
583 |
| - | |
584 |
| - | |
585 |
| - | |
586 |
| - | |
587 | 583 |
| |
588 |
| - | |
589 |
| - | |
590 |
| - | |
591 |
| - | |
592 |
| - | |
593 |
| - | |
594 |
| - | |
595 |
| - | |
596 |
| - | |
597 | 584 |
| |
598 | 585 |
| |
599 | 586 |
| |
|
Lines changed: 0 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
117 | 117 |
| |
118 | 118 |
| |
119 | 119 |
| |
120 |
| - | |
121 | 120 |
| |
122 | 121 |
| |
123 | 122 |
| |
| |||
153 | 152 |
| |
154 | 153 |
| |
155 | 154 |
| |
156 |
| - | |
157 | 155 |
| |
158 | 156 |
| |
159 | 157 |
| |
|
0 commit comments
Comments
(0)