Favor doc_reconstruct when reconstruct_document is also mentioned

[dhruvkaliraman7]

No description provided.

PR Overview

This PR modifies the OpenSearch reader initialization to favor the use of a document reconstructor over a boolean flag when both are provided.

• Added a condition to disable the reconstruct_document flag when a doc_reconstructor is set.
• Updated logging to inform users which configuration is being used.

Reviewed Changes

File	Description
lib/sycamore/sycamore/connectors/opensearch/opensearch_reader.py	Added a conditional block that logs and disables the reconstruct_document flag when a doc_reconstructor is provided to favor the latter.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

lib/sycamore/sycamore/connectors/opensearch/opensearch_reader.py:284

• There is a naming inconsistency between 'reconstruct_document' and 'doc_reconstructor'. Consider aligning the variable names (e.g., using 'doc_reconstruct') to clearly reflect that the document reconstructor is favored when both are set.

if query_params.reconstruct_document and query_params.doc_reconstructor is not None: