Re: BitmapHeapScan streaming read user and prelim refactoring
Tomas Vondra <tomas.vondra@enterprisedb.com>
Attachments
- v22b-0001-table_scan_bitmap_next_block-counts-lossy-and-e.patch (text/x-patch) patch 0001
- v22b-0002-review.patch (text/x-patch) patch 0002
- v22b-0003-Remove-table_scan_bitmap_next_tuple-parameter-t.patch (text/x-patch) patch 0003
- v22b-0004-Make-table_scan_bitmap_next_block-async-friendl.patch (text/x-patch) patch 0004
- v22b-0005-review.patch (text/x-patch) patch 0005
- v22b-0006-Add-common-interface-for-TBMIterators.patch (text/x-patch) patch 0006
- v22b-0007-review.patch (text/x-patch) patch 0007
- v22b-0008-BitmapHeapScan-initialize-some-prefetch-state-e.patch (text/x-patch) patch 0008
- v22b-0009-BitmapHeapScan-uses-unified-iterator.patch (text/x-patch) patch 0009
- v22b-0010-review.patch (text/x-patch) patch 0010
- v22b-0011-BitmapHeapScan-Make-prefetch-sync-error-more-de.patch (text/x-patch) patch 0011
- v22b-0012-Push-current-scan-descriptor-into-specialized-s.patch (text/x-patch) patch 0012
- v22b-0013-review.patch (text/x-patch) patch 0013
- v22b-0014-Remove-ss_current-prefix-from-ss_currentScanDes.patch (text/x-patch) patch 0014
- v22b-0015-Add-scan_in_progress-to-BitmapHeapScanState.patch (text/x-patch) patch 0015
- v22b-0016-Make-new-bitmap-table-scan-and-bitmap-heap-scan.patch (text/x-patch) patch 0016
- v22b-0017-review.patch (text/x-patch) patch 0017
- v22b-0018-Update-variable-names-in-bitmap-scan-descriptor.patch (text/x-patch) patch 0018
- v22b-0019-review.patch (text/x-patch) patch 0019
- v22b-0020-BitmapHeapScan-lift-and-shift-prefetch-function.patch (text/x-patch) patch 0020
- v22b-0021-BitmapHeapScan-Push-prefetch-code-into-heap-AM.patch (text/x-patch) patch 0021
- v22b-0022-review.patch (text/x-patch) patch 0022
- v22b-0023-BitmapHeapScan-move-per-scan-setup-into-a-helpe.patch (text/x-patch) patch 0023
- v22b-0024-Remove-table-AM-callback-scan_bitmap_next_block.patch (text/x-patch) patch 0024
- v22b-0025-review.patch (text/x-patch) patch 0025
- v22b-0026-Hard-code-TBMIterateResult-offsets-array-size.patch (text/x-patch) patch 0026
- v22b-0027-review.patch (text/x-patch) patch 0027
- v22b-0028-Separate-TBMIterator-and-TBMIterateResult.patch (text/x-patch) patch 0028
- v22b-0029-Use-streaming-I-O-in-Bitmap-Heap-Scans.patch (text/x-patch) patch 0029
Hi, I went through v22 to remind myself of what the patches do and do some basic review - I have some simple questions / comments for now, nothing major. I've kept the comments in separate 'review' patches, it does not seem worth copying here. regards -- Tomas Vondra EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
Commits
-
Fix bitmapheapscan incorrect recheck of NULL tuples
- aea916fe555a 18.0 landed
-
Increase default maintenance_io_concurrency to 16
- cc6be07ebde2 18.0 landed
-
Separate TBM[Shared|Private]Iterator and TBMIterateResult
- 944e81bf99db 18.0 landed
-
Improve read_stream.c advice for dense streams.
- 7ea8cd15661e 18.0 landed
-
Increase default effective_io_concurrency to 16
- ff79b5b2aba0 18.0 landed
-
Delay extraction of TIDBitmap per page offsets
- bfe56cdf9a4e 18.0 landed
-
Add lossy indicator to TBMIterateResult
- b8778c4cd8bc 18.0 landed
-
Move BitmapTableScan per-scan setup into a helper
- a5358c14b2fe 18.0 landed
-
Add and use BitmapHeapScanDescData struct
- f7a8fc10ccb8 18.0 landed
-
Fix bitmap table scan crash on iterator release
- 754c610e13b8 18.0 landed
-
Bitmap Table Scans use unified TBMIterator
- 1a0da347a7ac 18.0 landed
-
Add common interface for TBMIterators
- 7f9d4187e7ba 18.0 landed
-
Make table_scan_bitmap_next_block() async-friendly
- de380a62b5da 18.0 landed
-
Move EXPLAIN counter increment to heapam_scan_bitmap_next_block
- 7bd7aa4d3067 18.0 landed
-
Refactor tidstore.c iterator buffering.
- f6bef362cac8 18.0 cited
-
BitmapHeapScan: Remove incorrect assert and reset field
- a3e6c6f92991 17.0 landed
-
Change BitmapAdjustPrefetchIterator to accept BlockNumber
- 92641d8d651e 17.0 landed
-
BitmapHeapScan: Use correct recheck flag for skip_fetch
- 1fdb0ce9b109 17.0 landed
-
BitmapHeapScan: Push skip_fetch optimization into table AM
- 04e72ed617be 17.0 landed
-
BitmapHeapScan: postpone setting can_skip_fetch
- fe1431e39cdd 17.0 landed
-
BitmapHeapScan: begin scan after bitmap creation
- 1577081e9614 17.0 landed
-
Fix EXPLAIN Bitmap heap scan to count pages with no visible tuples
- f3e4581acdc8 12.19 landed
- 992189a3e94d 13.15 landed
- 262757b73286 14.12 landed
- d3d95f583995 15.7 landed
- 1f4eb734200a 16.3 landed
- 0960ae1967d0 17.0 landed
-
Remove redundant snapshot copying from parallel leader to workers
- 84c18acaf690 17.0 landed
-
Remove some obsolete smgrcloseall() calls.
- 6a8ffe812d19 17.0 cited
-
Remove the "snapshot too old" feature.
- f691f5b80a85 17.0 cited
-
Compute XID horizon for page level index vacuum on primary.
- 558a9165e081 12.0 cited