chore(server): Remove outdated TODO
Real-life experience has shown that the weighting of the metric produced here is appropriate.
This commit is contained in:
parent
904c3dade0
commit
2d4a3ea307
1 changed files with 1 additions and 5 deletions
|
@ -300,11 +300,7 @@ func groupLayer(dt *flow.DominatorTree, root *closure) Layer {
|
|||
sort.Strings(contents)
|
||||
|
||||
return Layer{
|
||||
Contents: contents,
|
||||
// TODO(tazjin): The point of this is to factor in
|
||||
// both the size and the popularity when making merge
|
||||
// decisions, but there might be a smarter way to do
|
||||
// it than a plain multiplication.
|
||||
Contents: contents,
|
||||
MergeRating: uint64(root.Popularity) * size,
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue