Merge pull request #788 from ahrtr/simply_free_20240708

ahrtr · web-flow · commit d72e6bf8c17e · 2024-07-22T12:54:30.000+01:00
No need to handle freelist as a specical case when freeing a page
diff --git a/internal/freelist/shared.go b/internal/freelist/shared.go
@@ -67,9 +67,6 @@ func (t *shared) Free(txid common.Txid, p *common.Page) {
 	allocTxid, ok := t.allocs[p.Id()]
 	if ok {
 		delete(t.allocs, p.Id())
-	} else if p.IsFreelistPage() {
-		// Freelist is always allocated by prior tx.
-		allocTxid = txid - 1
 	}
 
 	for id := p.Id(); id <= p.Id()+common.Pgid(p.Overflow()); id++ {

Original file line number	Diff line number	Diff line change
`@@ -67,9 +67,6 @@ func (t shared) Free(txid common.Txid, p common.Page) {`
`67`	`67`	`allocTxid, ok := t.allocs[p.Id()]`
`68`	`68`	`if ok {`
`69`	`69`	`delete(t.allocs, p.Id())`
`70`		`- } else if p.IsFreelistPage() {`
`71`		`- // Freelist is always allocated by prior tx.`
`72`		`- allocTxid = txid - 1`
`73`	`70`	`}`
`74`	`71`
`75`	`72`	`for id := p.Id(); id <= p.Id()+common.Pgid(p.Overflow()); id++ {`