[
https://issues.apache.org/jira/browse/YUNIKORN-2181?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
TingYao Huang updated YUNIKORN-2181:
------------------------------------
Target Version: 1.6.0 (was: 1.5.0)
> remove named returns in checkQueueHierarchyForPlacement
> -------------------------------------------------------
>
> Key: YUNIKORN-2181
> URL: https://issues.apache.org/jira/browse/YUNIKORN-2181
> Project: Apache YuniKorn
> Issue Type: Improvement
> Components: core - common
> Reporter: Wilfred Spiegelenburg
> Assignee: Chen, Kai-Chun
> Priority: Minor
> Labels: newbie
>
> In the {{pkg/common/configs/configvalidator.go}} the
> {{checkQueueHierarchyForPlacement}} function uses named return values. This
> can easily lead to difficult to diagnose bugs specially with the number of
> naked returns and the length of the function.
> We should remove the named return values and the naked returns.
> Note the linter "nakedret" complains about this when you run a simple
> {{./tools/golangci-lint run}}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
