[To dev/1.3] Subscription: Fixed multiple bugs (#17563)

[Caideyipi]

Description

As the title said.

---

This PR has:

• been self-reviewed.
  • concurrent read
  • concurrent write
  • concurrent read and write
• added documentation for new or modified features or behaviors.
• added Javadocs for most classes and all non-trivial methods.
• added or updated version, license, or notice information
• added comments explaining the "why" and the intent of the code wherever would not be obvious
  for an unfamiliar reader.
• added unit tests or modified existing tests to cover new code paths, ensuring the threshold
  for code coverage.
• added integration tests.
• been tested in a test IoTDB cluster.

---

Key changed/added classes (or packages if there are too many classes) in this PR

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.2% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.69%. Comparing base (04e6684) to head (8793b69).

Additional details and impacted files

@@              Coverage Diff              @@
##             dev/1.3   #17599      +/-   ##
=============================================
+ Coverage      41.62%   41.69%   +0.06%     
- Complexity       234      241       +7     
=============================================
  Files           3610     3610              
  Lines         238026   238030       +4     
  Branches       28892    28890       -2     
=============================================
+ Hits           99086    99246     +160     
+ Misses        138940   138784     -156     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.