forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitc584781
committed
Use pgBufferUsage for buffer usage tracking in analyze.
Previously, (auto)analyze used global variables VacuumPageHit,VacuumPageMiss, and VacuumPageDirty to track buffer usage. However,pgBufferUsage provides a more generic way to track buffer usage withsupport functions.This change replaces those global variables with pgBufferUsage inanalyze. Since analyze was the sole user of those variables, itremoves their declarations. Vacuum previously used those variables butreplaced them with pgBufferUsage as part of a bug fix, commit5cd72cc.Additionally, it adjusts the buffer usage message in both vacuum andanalyze for better consistency.Author: Anthonin BonnefoyReviewed-by: Masahiko Sawada, Michael PaquierDiscussion:https://postgr.es/m/CAO6_Xqr__kTTCLkftqS0qSCm-J7_xbRG3Ge2rWhucxQJMJhcRA%40mail.gmail.com1 parent2488058 commitc584781
File tree
7 files changed
+38
-43
lines changed- src
- backend
- access/heap
- commands
- storage/buffer
- utils/init
- include
7 files changed
+38
-43
lines changedLines changed: 16 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
608 | 608 |
| |
609 | 609 |
| |
610 | 610 |
| |
| 611 | + | |
| 612 | + | |
| 613 | + | |
611 | 614 |
| |
612 | 615 |
| |
613 | 616 |
| |
614 | 617 |
| |
615 | 618 |
| |
616 | 619 |
| |
617 | 620 |
| |
| 621 | + | |
| 622 | + | |
| 623 | + | |
| 624 | + | |
| 625 | + | |
| 626 | + | |
| 627 | + | |
618 | 628 |
| |
619 | 629 |
| |
620 | 630 |
| |
| |||
740 | 750 |
| |
741 | 751 |
| |
742 | 752 |
| |
743 |
| - | |
| 753 | + | |
744 | 754 |
| |
745 |
| - | |
| 755 | + | |
746 | 756 |
| |
747 | 757 |
| |
748 | 758 |
| |
749 | 759 |
| |
750 | 760 |
| |
751 |
| - | |
752 |
| - | |
753 |
| - | |
754 |
| - | |
| 761 | + | |
| 762 | + | |
| 763 | + | |
| 764 | + | |
755 | 765 |
| |
756 | 766 |
| |
757 | 767 |
| |
|
Lines changed: 22 additions & 19 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
303 | 303 |
| |
304 | 304 |
| |
305 | 305 |
| |
306 |
| - | |
307 |
| - | |
308 |
| - | |
| 306 | + | |
| 307 | + | |
309 | 308 |
| |
310 | 309 |
| |
311 | 310 |
| |
| |||
736 | 735 |
| |
737 | 736 |
| |
738 | 737 |
| |
| 738 | + | |
| 739 | + | |
| 740 | + | |
739 | 741 |
| |
740 |
| - | |
741 |
| - | |
742 |
| - | |
743 |
| - | |
744 |
| - | |
745 |
| - | |
746 |
| - | |
747 |
| - | |
| 742 | + | |
| 743 | + | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
| 747 | + | |
| 748 | + | |
| 749 | + | |
| 750 | + | |
748 | 751 |
| |
749 | 752 |
| |
750 | 753 |
| |
| |||
770 | 773 |
| |
771 | 774 |
| |
772 | 775 |
| |
773 |
| - | |
774 |
| - | |
775 |
| - | |
776 |
| - | |
| 776 | + | |
| 777 | + | |
| 778 | + | |
| 779 | + | |
777 | 780 |
| |
778 | 781 |
| |
779 | 782 |
| |
| |||
796 | 799 |
| |
797 | 800 |
| |
798 | 801 |
| |
799 |
| - | |
800 |
| - | |
801 |
| - | |
802 |
| - | |
| 802 | + | |
| 803 | + | |
| 804 | + | |
| 805 | + | |
803 | 806 |
| |
804 | 807 |
| |
805 | 808 |
| |
|
Lines changed: 0 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
603 | 603 |
| |
604 | 604 |
| |
605 | 605 |
| |
606 |
| - | |
607 |
| - | |
608 |
| - | |
609 | 606 |
| |
610 | 607 |
| |
611 | 608 |
| |
|
Lines changed: 0 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1043 | 1043 |
| |
1044 | 1044 |
| |
1045 | 1045 |
| |
1046 |
| - | |
1047 |
| - | |
1048 |
| - | |
1049 | 1046 |
| |
1050 | 1047 |
| |
1051 | 1048 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1192 | 1192 |
| |
1193 | 1193 |
| |
1194 | 1194 |
| |
1195 |
| - | |
1196 | 1195 |
| |
1197 | 1196 |
| |
1198 | 1197 |
| |
| |||
1588 | 1587 |
| |
1589 | 1588 |
| |
1590 | 1589 |
| |
1591 |
| - | |
1592 | 1590 |
| |
1593 | 1591 |
| |
1594 | 1592 |
| |
| |||
2582 | 2580 |
| |
2583 | 2581 |
| |
2584 | 2582 |
| |
2585 |
| - | |
2586 | 2583 |
| |
2587 | 2584 |
| |
2588 | 2585 |
| |
| |||
5122 | 5119 |
| |
5123 | 5120 |
| |
5124 | 5121 |
| |
5125 |
| - | |
5126 | 5122 |
| |
5127 | 5123 |
| |
5128 | 5124 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
153 | 153 |
| |
154 | 154 |
| |
155 | 155 |
| |
156 |
| - | |
157 |
| - | |
158 |
| - | |
159 |
| - | |
160 | 156 |
| |
161 | 157 |
| |
162 | 158 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
284 | 284 |
| |
285 | 285 |
| |
286 | 286 |
| |
287 |
| - | |
288 |
| - | |
289 |
| - | |
290 |
| - | |
291 | 287 |
| |
292 | 288 |
| |
293 | 289 |
| |
|
0 commit comments
Comments
(0)