[YUNIKORN-2285] Don't re-calculate reservationKey

[pbacsko]

What is this PR for?

Don't constantly re-calculate reservationKey in case of reserved allocations. We keep generating this string if the cluster is busy.

What type of PR is it?

• - Bug Fix
• - Improvement
• - Feature
• - Documentation
• - Hot Fix
• - Refactoring

Todos

• - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-2285

How should this be tested?

Screenshots (if appropriate)

Questions:

• - The licenses files need update.
• - There is breaking changes for older versions.
• - It needs documentation.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (59cda41) 77.76% compared to head (1f4b51c) 77.80%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #766      +/-   ##
==========================================
+ Coverage   77.76%   77.80%   +0.03%     
==========================================
  Files          82       82              
  Lines       13436    13457      +21     
==========================================
+ Hits        10449    10470      +21     
  Misses       2660     2660              
  Partials      327      327              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zhuqi-lucas]

LGTM

[craigcondit]

+1 LGTM.