JakeHillion/storj
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

satellite/metabase: exclude expired segments from ListVerifySegments results

Browse Source 
I believe that this change in semantics won't break anything, because
ListVerifySegments is only used by cmd/tools/segment-verify (which only
needs to operate on non-expired segments) and various tests, none of
which expect ListVerifySegments to include expired segments.

Change-Id: I037f43b16bc5750ed914bc32949418e001df1a8c
This commit is contained in:
paul cannon 2023-10-13 16:15:58 -05:00 committed by Storj Robot
parent 23c5d6c287
commit 58f75502f3
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
satellite/metabase/list_verify_segments.go
View File

@ -61,7 +61,8 @@ func (opts *ListVerifySegments) getQueryAndParameters(asof string) (string, []in
WHERE
(stream_id, position) > ($1, $2) AND
inline_data IS NULL AND
remote_alias_pieces IS NOT NULL
remote_alias_pieces IS NOT NULL AND
(segments.expires_at IS NULL OR segments.expires_at > now())
ORDER BY stream_id ASC, position ASC
LIMIT $3
`, []interface{}{
@ -82,7 +83,8 @@ func (opts *ListVerifySegments) getQueryAndParameters(asof string) (string, []in
stream_id = ANY($1) AND
(segments.stream_id, segments.position) > ($2, $3) AND
segments.inline_data IS NULL AND
segments.remote_alias_pieces IS NOT NULL
segments.remote_alias_pieces IS NOT NULL AND
(segments.expires_at IS NULL OR segments.expires_at > now())
ORDER BY segments.stream_id ASC, segments.position ASC
LIMIT $4
`, []interface{}{