Fix the getLastConfirmedEntry order

managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/cache/ReadEntryUtils.java

@@ -28,18 +28,19 @@ class ReadEntryUtils {
 
     static CompletableFuture<LedgerEntries> readAsync(ManagedLedger ml, ReadHandle handle, long firstEntry,
                                                       long lastEntry) {
-        final var lastConfirmedEntry = ml.getLastConfirmedEntry();
-        if (lastConfirmedEntry == null) {
-            return CompletableFuture.failedFuture(new ManagedLedgerException(
-                    "LastConfirmedEntry is null when reading ledger " + handle.getId()));
-        }
         if (ml.getOptionalLedgerInfo(handle.getId()).isEmpty()) {
             // The read handle comes from another managed ledger, in this case, we can only compare the entry range with
             // the LAC of that read handle. Specifically, it happens when this method is called by a
             // ReadOnlyManagedLedgerImpl object.
             return handle.readAsync(firstEntry, lastEntry);
         }
-        // Compare the entry range with the lastConfirmedEntry maintained by the managed ledger
+        // Compare the entry range with the lastConfirmedEntry maintained by the managed ledger because the entry cache
+        // of `ShadowManagedLedgerImpl` reads entries via `ReadOnlyLedgerHandle`, which never updates `lastAddConfirmed`
+        final var lastConfirmedEntry = ml.getLastConfirmedEntry();
+        if (lastConfirmedEntry == null) {
+            return CompletableFuture.failedFuture(new ManagedLedgerException(
+                    "LastConfirmedEntry is null when reading ledger " + handle.getId()));
+        }
         if (handle.getId() > lastConfirmedEntry.getLedgerId()) {
             return CompletableFuture.failedFuture(new ManagedLedgerException("LastConfirmedEntry is "
                     + lastConfirmedEntry + " when reading ledger " + handle.getId()));