feature/4-add_executor #13
No reviewers
TrueCloudLab/storage-services-developers
Labels
No labels
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-s3-lifecycler#13
Loading…
Reference in a new issue
No description provided.
Delete branch "dkirillov/frostfs-s3-lifecycler:feature/4-add_executor"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
close #4
7539f43a9a
to4bdde21ccf
WIP: feature/4-add_executorto feature/4-add_executorLooks great overall! I would like to add some more tests for match-functions but we can do it later.
@ -120,0 +116,4 @@
// // it provides compatibility with previous tree service api where
// // single uint64(0) value is dropped from signature
// poolPrm.RootID = nil
//}
Do you want to keep it in the code or just forgot to remove?
@ -0,0 +106,4 @@
})
if err != nil {
wg.Done()
I don't mind having two
wg.Done()
but it seems like we can do it once in the end of case statement.Actually we cannot. We have to invoke
wg.Done()
after task is done. End ofSubmit
doesn't guaranties that task was ended.Yep, missed that it is non-blocking operation.
@ -0,0 +140,4 @@
if ni.CurrentEpoch() != job.Epoch {
ni.SetCurrentEpoch(job.Epoch)
}
Is it worth a warning?
@ -0,0 +203,4 @@
if len(matchers) == 0 {
return nil, errNotApplicableRule
}
Is it an error, actually?
This function returns error, then it is returned by
abortMultiparts()
and then logged as a WARN inworker()
.Maybe we should ignore this error in the
worker()
but I don't see any usage of this error in this case.Sometimes it can be useful to see that we don't do anything despite rules are not empty. But I can add ignoring this error in
worker
4bdde21ccf
to15c9d55b03
@ -0,0 +198,4 @@
if err != nil && !errors.Is(err, errNotApplicableRule) {
return nil, err
}
Seems that in case of
errNotApplicableRule
we can't usematchFn
because it's nil127816c66a
tob3374bb565