Remove unnecessary intermediate String deserialization of document bodies

[emilienbev]

Fixes #2121

• You have read the Spring Data contribution guidelines.
• There is a ticket in the bug tracker for the project in our JIRA.
• You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
• You submit test cases (unit or integration tests) that back your changes.
• You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

[Copilot]

Pull request overview

This PR addresses issue #2121 by eliminating an unnecessary intermediate String conversion when decoding KV document bodies, enabling decoding directly from raw response bytes (notably for reactive KV operations) while keeping string-based decoding available.

Changes:

• Add byte[] decode paths across template support APIs and internal decode flow (TemplateSupport / ReactiveTemplateSupport → AbstractTemplateSupport → TranslationService).
• Update reactive KV operations (e.g., find-by-id, range scan, replicas) to pass contentAsBytes() instead of converting to String.
• Extend Jackson translation to decode directly from byte[] and add tests validating byte-based decoding (including non-ASCII content).

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/test/java/org/springframework/data/couchbase/repository/query/ReactiveCouchbaseRepositoryQueryCollectionIntegrationTests.java	Adjusts reactive test flow to block for completion (avoids async subscribe).
src/test/java/org/springframework/data/couchbase/core/convert/translation/JacksonTranslationServiceTests.java	Adds coverage for decoding from UTF-8 bytes (ASCII + non-ASCII).
src/main/java/org/springframework/data/couchbase/core/convert/translation/TranslationService.java	Adds decode(byte[], ...) default overload to support byte-based decoding.
src/main/java/org/springframework/data/couchbase/core/convert/translation/JacksonTranslationService.java	Implements direct byte[] parsing via Jackson JsonParser.
src/main/java/org/springframework/data/couchbase/core/TemplateSupport.java	Adds byte[] overload for entity decoding (sync template support).
src/main/java/org/springframework/data/couchbase/core/ReactiveTemplateSupport.java	Adds byte[] overload for entity decoding (reactive template support).
src/main/java/org/springframework/data/couchbase/core/ReactiveRangeScanOperationSupport.java	Uses contentAsBytes() to avoid intermediate String creation.
src/main/java/org/springframework/data/couchbase/core/ReactiveFindFromReplicasByIdOperationSupport.java	Uses contentAsBytes() for replica reads.
src/main/java/org/springframework/data/couchbase/core/ReactiveFindByIdOperationSupport.java	Uses contentAsBytes() across get variants (touch/lock/get/tx path).
src/main/java/org/springframework/data/couchbase/core/ReactiveCouchbaseTemplateSupport.java	Implements reactive byte[] decodeEntity overload.
src/main/java/org/springframework/data/couchbase/core/NonReactiveSupportWrapper.java	Bridges reactive byte[] decode to non-reactive support.
src/main/java/org/springframework/data/couchbase/core/CouchbaseTemplateSupport.java	Implements sync byte[] decodeEntity overload.
src/main/java/org/springframework/data/couchbase/core/AbstractTemplateSupport.java	Adds byte[] decodeEntityBase path and refactors shared decode/finalize steps.
.gitignore	Ignores src/test/resources/integration.local.properties.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[onobc]

👋🏻 @emilienbev
Great work! Thank you for the excellent detail in the issue and the PR description.

I have one request that spans multiple files... do you mind moving your @ author... tags to the end of the list as we typically add in history order (i.e. the original author of the source is the first entry).

[onobc]

Nice work @emilienbev - looks good to me. I have a comment around logging and using String#formatted but you can do what you want w/ that one.

The only other comment I have was mentioned by Mark - we typically don't add final to anything other immutable fields that are set in the constructor.