Uploaded image for project: 'Apache YuniKorn'
  1. Apache YuniKorn
  2. YUNIKORN-2137

Remove defer cleanup in checkLimits

    XMLWordPrintableJSON

Details

    Description

      Code Reference: https://github.com/apache/yunikorn-core/blob/30e69f97585bc3a82f4b9c54f4cc4b601120a776/pkg/common/configs/configvalidator.go#L586-L594

      Slack Reference: https://yunikornworkspace.slack.com/archives/CL9CRJ1KM/p1699430436233019

       

      The existedUserName and existedGroupName in checkLimits will be GC after the goroutine leaves the function. It's safe to remove the defer function.

      Attachments

        Issue Links

          Activity

            People

              targetoee JiaChi Wang
              yangpoan PoAn Yang
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: