[ISSUE-376] Fix concurrency problems may occur when the ApplicationManager register app

[smallzhongfeng]

What changes were proposed in this pull request?

Lock when reading and writing the map appAndTimes.

Why are the changes needed?

To resolve #376 .

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Passed origin uts.

[jerqi]

cc @leixm Could you help me review this patch?
If you feel that this patch is ok, you approve this pr and say LGTM. (LGTM means that look good to me)

[codecov-commenter]

Codecov Report

Merging #382 (c21fde6) into master (57a834b) will increase coverage by 0.17%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##             master     #382      +/-   ##
============================================
+ Coverage     58.46%   58.63%   +0.17%     
- Complexity     1572     1590      +18     
============================================
  Files           193      193              
  Lines         10846    10920      +74     
  Branches        954      958       +4     
============================================
+ Hits           6341     6403      +62     
- Misses         4131     4138       +7     
- Partials        374      379       +5     

Impacted Files	Coverage Δ
...apache/uniffle/coordinator/ApplicationManager.java	84.11% <33.33%> (-1.35%)	⬇️
...apache/uniffle/coordinator/AccessQuotaChecker.java	94.73% <85.71%> (-1.10%)	⬇️
...a/org/apache/uniffle/coordinator/QuotaManager.java	89.55% <100.00%> (+7.55%)	⬆️
...a/org/apache/uniffle/common/ShuffleServerInfo.java	75.00% <0.00%> (-25.00%)	⬇️
...orage/handler/impl/LocalFileClientReadHandler.java	59.37% <0.00%> (-6.15%)	⬇️
...va/org/apache/uniffle/client/util/ClientUtils.java	52.27% <0.00%> (-5.23%)	⬇️
.../java/org/apache/uniffle/common/util/RssUtils.java	60.58% <0.00%> (-2.78%)	⬇️
...le/storage/handler/impl/HdfsClientReadHandler.java	75.00% <0.00%> (-1.93%)	⬇️
...che/uniffle/client/impl/ShuffleReadClientImpl.java	88.46% <0.00%> (-1.64%)	⬇️
.../java/org/apache/uniffle/server/ShuffleServer.java	67.39% <0.00%> (-1.03%)	⬇️
... and 25 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[smallzhongfeng]

cc @jerqi @leixm

[jerqi]

LGTM, @leixm Do you have another suggestion?

[leixm]

+1 LGTM.

[jerqi]

@leixm You could click the approve button.

[jerqi]

Merged. thanks @leixm @smallzhongfeng

[smallzhongfeng]

Thanks all. @jerqi @leixm