[fix] [broker] Subscription stuck due to called Admin API analyzeSubscriptionBacklog

[poorbarcode]

Motivation

The Admin API analyzeSubscriptionBacklog will create an OpReadEntry and trigger a read, which leads Managecursor.readPosition to move forward. It makes the subscription stuck(just like the consumer receives messages but does not acknowledge them).

Modifications

• Substitute new non-durable cursor for subscription.cursor
• Rename the param startCursorPosition of new NonDurableCursor to mdPosition, make the code more clearer

Documentation

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

Matching PR in forked repository

PR in forked repository: x

[poorbarcode]

A new update: The new Non-Durable cursor has not copied the ack state when creating, I will fix it tomorrow.

New update(2024-02-05): fixed

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 64.70588% with 12 lines in your changes missing coverage. Please review.

Project coverage is 73.56%. Comparing base (fce0717) to head (bd9869c).
Report is 1255 commits behind head on master.

Files with missing lines	Patch %	Lines
...che/bookkeeper/mledger/impl/ManagedCursorImpl.java	62.50%	4 Missing and 2 partials ⚠️
...ker/service/persistent/PersistentSubscription.java	70.58%	5 Missing ⚠️
...lsar/common/util/collections/BitSetRecyclable.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #22019      +/-   ##
============================================
+ Coverage     72.90%   73.56%   +0.66%     
- Complexity    32464    32559      +95     
============================================
  Files          1850     1874      +24     
  Lines        138611   139253     +642     
  Branches      15201    15263      +62     
============================================
+ Hits         101052   102444    +1392     
+ Misses        29711    28888     -823     
- Partials       7848     7921      +73     

Flag	Coverage Δ
inttests	24.63% <0.00%> (?)
systests	24.32% <0.00%> (?)
unittests	72.85% <64.70%> (-0.05%)	⬇️

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

Files with missing lines	Coverage Δ
...lsar/common/util/collections/BitSetRecyclable.java	46.01% <0.00%> (-0.13%)	⬇️
...ker/service/persistent/PersistentSubscription.java	76.54% <70.58%> (-0.37%)	⬇️
...che/bookkeeper/mledger/impl/ManagedCursorImpl.java	78.84% <62.50%> (-0.61%)	⬇️

... and 202 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.