Commit18b45c2

committed

delete in batches

1 parent1df93be commit18b45c2Copy full SHA for 18b45c2

File tree

+39

-3

lines changed

+39

-3

lines changed

Lines changed: 21 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -1526,6 +1526,15 @@ func (q *FakeQuerier) DeleteOldWorkspaceAgentStats(_ context.Context) error {`
`1526`	`1526`	`)`
`1527`	`1527`	`FROM`
`1528`	`1528`	`template_usage_stats`
	`1529`	`+)`
	`1530`	`+AND created_at < (`
	`1531`	`+-- Delete at most in batches of 3 days (with a batch size of 3 days, we`
	`1532`	`+-- can clear out the previous 6 months of data in ~60 iterations) whilst`
	`1533`	`+-- keeping the DB load relatively low.`
	`1534`	`+SELECT`
	`1535`	`+COALESCE(MIN(created_at) + '3 days'::interval, NOW())`
	`1536`	`+FROM`
	`1537`	`+workspace_agent_stats`
`1529`	`1538`	`);`
`1530`	`1539`	`*/`
`1531`	`1540`
`@@ -1543,10 +1552,19 @@ func (q *FakeQuerier) DeleteOldWorkspaceAgentStats(_ context.Context) error {`
`1543`	`1552`	`}`
`1544`	`1553`
`1545`	`1554`	`varvalidStats []database.WorkspaceAgentStat`
	`1555`	`+varbatchLimit time.Time`
	`1556`	`+for_,stat:=rangeq.workspaceAgentStats {`
	`1557`	`+ifbatchLimit.IsZero()\|\|stat.CreatedAt.Before(batchLimit) {`
	`1558`	`+batchLimit=stat.CreatedAt`
	`1559`	`+}`
	`1560`	`+}`
	`1561`	`+ifbatchLimit.IsZero() {`
	`1562`	`+batchLimit=time.Now()`
	`1563`	`+}else {`
	`1564`	`+batchLimit=batchLimit.AddDate(0,0,3)`
	`1565`	`+}`
`1546`	`1566`	`for_,stat:=rangeq.workspaceAgentStats {`
`1547`		`-fmt.Println(stat.CreatedAt,limit)`
`1548`		`-ifstat.CreatedAt.Before(limit) {`
`1549`		`-fmt.Println("delete")`
	`1567`	`+ifstat.CreatedAt.Before(limit)&&stat.CreatedAt.Before(batchLimit) {`
`1550`	`1568`	`continue`
`1551`	`1569`	`}`
`1552`	`1570`	`validStats=append(validStats,stat)`

Lines changed: 9 additions & 0 deletions

Some generated files are not rendered by default. Learn more aboutcustomizing how changed files appear on GitHub.

Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,15 @@ WHERE`
`109`	`109`	`)`
`110`	`110`	`FROM`
`111`	`111`	`template_usage_stats`
	`112`	`+)`
	`113`	`+AND created_at< (`
	`114`	`+-- Delete at most in batches of 3 days (with a batch size of 3 days, we`
	`115`	`+-- can clear out the previous 6 months of data in ~60 iterations) whilst`
	`116`	`+-- keeping the DB load relatively low.`
	`117`	`+SELECT`
	`118`	`+COALESCE(MIN(created_at)+'3 days'::interval, NOW())`
	`119`	`+FROM`
	`120`	`+workspace_agent_stats`
`112`	`121`	`);`
`113`	`122`
`114`	`123`	`-- name: GetDeploymentWorkspaceAgentStats :one`

Comments

(0)