object: Introduce new fields for ECHeader
type #223
Labels
No Label
P0
P1
P2
P3
good first issue
pool
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-sdk-go#223
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "aarifullin/frostfs-sdk-go:feat/ecHeader_parentSplit"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
ec_parent
search filter;parentSplitID
,parentSplitParentID
fields forECHeader
;Split
andReconstruct
;6d43e166ed
to3c78222892
@ -27,0 +32,4 @@
chld := obj.Parent().Children()
for i := range chld {
if chld[i].Equals(objID) {
parentSplitIndex = uint32(i)
@fyrchik, please, could you fix me if this is an incorrect way to define the index within big Split?
These
Children
are not always present (because when we cut 2nd child we do not now how much more to expect).I believe the only way to do this is to accept an explicit parameter in
Split
Maybe even another public function for this which reuses most of the code, so that we have a simple interface for small objects.
Maybe even another public function for this which reuses most of the code, so that we have a simple interface for small objects.
3c78222892
to8ccb0451be
8ccb0451be
toa4e80d3f6c
@ -0,0 +4,4 @@
"context"
"fmt"
apemanager_v2 "git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/apemanager"
In other places we are following LowerCamelCase convention.
This commit with these changes has been picked accidentally :) I've removed it
object: Introduce new fields for `ECHeader` typeto WIP: object: Introduce new fields for `ECHeader` typea4e80d3f6c
to3d9313b323
3d9313b323
to729641b285
729641b285
to340c72526d
340c72526d
toee00a4da10
ee00a4da10
tofacf1f2a1e
facf1f2a1e
toea4585bea8
ea4585bea8
to3de256d05e
WIP: object: Introduce new fields for `ECHeader` typeto object: Introduce new fields for `ECHeader` type@ -0,0 +9,4 @@
"git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/rpc/client"
session_v2 "git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/session"
"git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/signature"
apemanager_sdk "git.frostfs.info/TrueCloudLab/frostfs-sdk-go/apemanager"
I believe sdk packages can use default names.
@ -17,1 +13,3 @@
headerLength uint32
parent oid.ID
parentSplitID *SplitID
parentSplitIndexSet bool
There should not be such variable: we always treat missing numbers as 0 in API and 0 also makes sense for small objects -- which are like big ones, but with only 1 part.