[ARCTIC-1018][AMS] Avoiding concurrent operations by tasksLock (#1022)

* [ARCTIC-1018][AMS] LinkedHashMap modified to ConcurrentHashMap

* Revert "[ARCTIC-1018][AMS] LinkedHashMap modified to ConcurrentHashMap"

This reverts commit 4b262fe.

* [ARCTIC-1018][AMS] Avoiding concurrent operations by `tasksLock`

ams/ams-server/src/main/java/com/netease/arctic/ams/server/optimize/TableOptimizeItem.java

@@ -858,12 +858,17 @@ public List<OptimizeTaskItem> getOptimizeTasksToExecute() {
    * If task execute timeout, set it to be Failed.
    */
   public void checkTaskExecuteTimeout() {
-    optimizeTasks.values().stream().filter(OptimizeTaskItem::executeTimeout)
-        .forEach(task -> {
-          task.onFailed(new ErrorMessage(System.currentTimeMillis(), "execute expired"),
-              System.currentTimeMillis() - task.getOptimizeRuntime().getExecuteTime());
-          LOG.error("{} execute timeout, change to Failed", task.getTaskId());
-        });
+    tasksLock.lock();
+    try {
+      optimizeTasks.values().stream().filter(OptimizeTaskItem::executeTimeout)
+          .forEach(task -> {
+            task.onFailed(new ErrorMessage(System.currentTimeMillis(), "execute expired"),
+                System.currentTimeMillis() - task.getOptimizeRuntime().getExecuteTime());
+            LOG.error("{} execute timeout, change to Failed", task.getTaskId());
+          });
+    } finally {
+      tasksLock.unlock();
+    }
   }
 
   /**