[fix](editlog) Fix BDBEnvironment.close() not holding writeLock causing ConcurrentModificationException

BDBEnvironment.close() iterates openedDatabases without acquiring
lock.writeLock(), while openDatabase() concurrently modifies the same
ArrayList under the write lock. This causes ConcurrentModificationException
when a RollbackException triggers close() on the replayer thread while
HTTP/heartbeat/RPC handler threads call openDatabase() via getMaxJournalId().

The race window: replayer catches RollbackException in getDatabaseNames(),
calls bdbEnvironment.close() which iterates openedDatabases with for-each.
Concurrently, a SHOW FRONTENDS or heartbeat thread calls getMaxJournalId()
→ openDatabase() → openedDatabases.add(). The ArrayList iterator detects
the structural modification and throws ConcurrentModificationException.
This kills the replayer thread, causing the follower FE to fall permanently
behind the master.

Fix: wrap the entire close() body with lock.writeLock() to mutually exclude
with openDatabase() and removeDatabase().

Author: dataroaring

fe/fe-core/src/main/java/org/apache/doris/journal/bdbje/BDBEnvironment.java

@@ -481,32 +481,37 @@ public List<Long> getDatabaseNames() {
 
     // Close the store and environment
     public void close() {
-        for (Database db : openedDatabases) {
-            try {
-                db.close();
-            } catch (DatabaseException exception) {
-                LOG.error("Error closing db {} will exit", db.getDatabaseName(), exception);
+        lock.writeLock().lock();
+        try {
+            for (Database db : openedDatabases) {
+                try {
+                    db.close();
+                } catch (DatabaseException exception) {
+                    LOG.error("Error closing db {} will exit", db.getDatabaseName(), exception);
+                }
             }
-        }
-        openedDatabases.clear();
+            openedDatabases.clear();
 
-        if (epochDB != null) {
-            try {
-                epochDB.close();
-                epochDB = null;
-            } catch (DatabaseException exception) {
-                LOG.error("Error closing db {} will exit", epochDB.getDatabaseName(), exception);
+            if (epochDB != null) {
+                try {
+                    epochDB.close();
+                    epochDB = null;
+                } catch (DatabaseException exception) {
+                    LOG.error("Error closing db {} will exit", epochDB.getDatabaseName(), exception);
+                }
             }
-        }
 
-        if (replicatedEnvironment != null) {
-            try {
-                // Finally, close the store and environment.
-                replicatedEnvironment.close();
-                replicatedEnvironment = null;
-            } catch (DatabaseException exception) {
-                LOG.error("Error closing replicatedEnvironment", exception);
+            if (replicatedEnvironment != null) {
+                try {
+                    // Finally, close the store and environment.
+                    replicatedEnvironment.close();
+                    replicatedEnvironment = null;
+                } catch (DatabaseException exception) {
+                    LOG.error("Error closing replicatedEnvironment", exception);
+                }
             }
+        } finally {
+            lock.writeLock().unlock();
         }
     }