remove named returns in checkQueueHierarchyForPlacement

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