Reapply "SolaceIO.Read: handle occasional cases when finalizeCheckpoint is not executed (#32962)" (#33259)

[bzablocki]

Reapply changes that were introduced and reverted in #32962. See the main discussion there.

---

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

• Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
• Update CHANGES.md with noteworthy changes.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

[bzablocki]

@ppawel FYI, reapplying changes here.
@Abacn sorry for the mixup, these are exactly the same changes as in #32962. Turns out the reason behind failing integration tests were my other changes from a different PR. Would you mind merging these changes? The review was in #32962.

[github-actions]

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

[bzablocki]

Run Java_Solace_IO_Direct PreCommit

[ppawel]

OK, thanks for the info @bzablocki , I am currently preparing a version for us with 2.61.0 and this PR applied. Is it planned on your side to do an official backport (2.61.1 or something like that)?

[bzablocki]

I'm afraid a patch to an already released version can only be applied in some very specific cases, specifically when there is a major performance issue in the new version, or some serious bug is introduced. I don't think this PR qualifies for that.

[bzablocki]

R: @Abacn
Would you mind merging these changes? This PR is exactly the same as #32962, which was reverted.
The reason behind failing integration tests were my other changes from a different PR. The review was in #32962.

[github-actions]

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment assign set of reviewers

[Abacn]

Thanks!