a-savchuk/frostfs-sdk-go
1
0
Fork 0
forked from TrueCloudLab/frostfs-sdk-go
Code Pull requests Activity

[#59] netmap: Remove unused param from getSelection()

Browse source 
Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2023-04-18 12:56:23 +03:00 committed by Evgenii Stratonikov
parent dce55a436a
commit 7c75db2f2d
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
netmap/netmap.go
View file

@ -189,7 +189,7 @@ func (m NetMap) ContainerNodes(p PlacementPolicy, pivot []byte) ([][]NodeInfo, e
s.SetCount(p.replicas[i].GetCount()) s.SetCount(p.replicas[i].GetCount())
s.SetFilter(mainFilterName) s.SetFilter(mainFilterName)
nodes, err := c.getSelection(p, s) nodes, err := c.getSelection(s)
if err != nil { if err != nil {
return nil, err return nil, err
} }

4
netmap/selector.go
View file

@ -23,7 +23,7 @@ func (c *context) processSelectors(p PlacementPolicy) error {
c.processedSelectors[sName] = &p.selectors[i] c.processedSelectors[sName] = &p.selectors[i]
result, err := c.getSelection(p, p.selectors[i]) result, err := c.getSelection(p.selectors[i])
if err != nil { if err != nil {
return err return err
} }
@ -56,7 +56,7 @@ func calcBucketWeight(ns nodes, a aggregator, wf weightFunc) float64 {
// getSelection returns nodes grouped by s.attribute. // getSelection returns nodes grouped by s.attribute.
// Last argument specifies if more buckets can be used to fulfill CBF. // Last argument specifies if more buckets can be used to fulfill CBF.
func (c *context) getSelection(p PlacementPolicy, s netmap.Selector) ([]nodes, error) { func (c *context) getSelection(s netmap.Selector) ([]nodes, error) {
bucketCount, nodesInBucket := calcNodesCount(s) bucketCount, nodesInBucket := calcNodesCount(s)
buckets := c.getSelectionBase(s) buckets := c.getSelectionBase(s)