Fixes #24849

Motivation

Detailed log as follows: https://gist.github.com/Denovo1998/0707fff19079a6694e4f7edbceddc10a

• Previously the test fenced the topic (by opening/closing the ManagedLedger externally) before the producer sent any message. If the current ledger is empty (LAC = -1), the broker’s recovery path deletes that empty ledger. Meanwhile, the managed ledger’s metadata update loop still tries to read/write that deleted ledger, leading to BKNoSuchLedgerExistsOnMetadataServerException and putting the ManagedLedger into a Fenced state.
• Recovery then fails to complete within the producer’s send timeout, so the first send blocks and times out, making the test flaky or consistently failing.
• By moving the fence after the first message is successfully written, the current ledger is non-empty, recovery proceeds by rolling to a new ledger instead of deleting the empty one, and the test can reliably verify the sequenceId behavior across a transient send error.

Modifications

• Reordered the test flow in SequenceIdWithErrorTest:
  • Create the producer and send the first message ("Hello-0") before fencing, ensuring the current ledger is not empty.
  • Then fence the topic by externally opening/closing the ManagedLedger (via ManagedLedgerClientFactory), which triggers broker recovery as intended.
  • Continue sending the remaining messages and verify their values and sequenceId.
• Added a timeout to consumer.receive(..., 10, TimeUnit.SECONDS) to avoid unbounded waits in abnormal conditions.
• Ensured resources are properly closed (ManagedLedger clientFactory.close() and eventLoopGroup.shutdownGracefully().get()), and the Pulsar client is closed at the end of the test.

Verifying this change

• Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

• Added integration tests for end-to-end deployment with large payloads (10MB)
• Extended integration test for recovery after broker failure

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository: Denovo1998#12