[fix] [broker] Let Pending ack handler can retry to init when encounters a metadata store error

[poorbarcode]

Motivation

The Pending Ack handler can not recover after encountering a MetadataStoreException even the metadata store has been re-established.

2024-07-24T02:56:11,347+0000 [pulsar-transaction-executor-5-1] ERROR org.apache.pulsar.broker.transaction.pendingack.impl.PendingAckHandleImpl - [persistent://{tenant}/{ns}/{topic}] [multiTopicsReader-5c010a1ec8] PendingAckHandleImpl init fail!
org.apache.pulsar.metadata.api.MetadataStoreException: org.apache.zookeeper.KeeperException$ConnectionLossException: KeeperErrorCode = ConnectionLoss for /managed-ledgers/{tenant}/{namespace}/persistent/{topic}-multiTopicsReader-5c010a1ec8__transaction_pending_ack
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) ~[io.netty-netty-common-4.1.108.Final.jar:4.1.108.Final]
	at org.apache.zookeeper.KeeperException.create(KeeperException.java:101)
	at org.apache.pulsar.broker.transaction.pendingack.impl.PendingAckHandleImpl.lambda$init$1(PendingAckHandleImpl.java:185) ~[io.streamnative-pulsar-broker-3.0.4.6.jar:3.0.4.6]
        ...

Modifications

Add MetadataStoreException and BKException into the retriable error list

Documentation

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

Matching PR in forked repository

PR in forked repository: x

[poorbarcode]

/pulsarbot rerun-failure-checks

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.56%. Comparing base (bbc6224) to head (08f0209).
Report is 780 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23153      +/-   ##
============================================
+ Coverage     73.57%   74.56%   +0.98%     
- Complexity    32624    33603     +979     
============================================
  Files          1877     1919      +42     
  Lines        139502   144268    +4766     
  Branches      15299    15777     +478     
============================================
+ Hits         102638   107570    +4932     
+ Misses        28908    28458     -450     
- Partials       7956     8240     +284     

Flag	Coverage Δ
inttests	27.90% <ø> (+3.32%)	⬆️
systests	24.73% <ø> (+0.41%)	⬆️
unittests	73.91% <ø> (+1.06%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...nsaction/pendingack/impl/PendingAckHandleImpl.java	72.97% <ø> (+2.12%)	⬆️

... and 504 files with indirect coverage changes