Skip to content

OAK-12053 | oak-search-elastic: set max analyzed offset for highlights#2681

Merged
fabriziofortino merged 6 commits intoapache:trunkfrom
fabriziofortino:OAK-12053
Jan 14, 2026
Merged

OAK-12053 | oak-search-elastic: set max analyzed offset for highlights#2681
fabriziofortino merged 6 commits intoapache:trunkfrom
fabriziofortino:OAK-12053

Conversation

@fabriziofortino
Copy link
Copy Markdown
Contributor

Longer fields would be truncated at runtime instead of throwing an error and fail the query.

@fabriziofortino fabriziofortino changed the title OAK-12053 | oak-search-elastic: set max analyzed offset for highlights. OAK-12053 | oak-search-elastic: set max analyzed offset for highlights Jan 10, 2026
Copy link
Copy Markdown
Contributor

@ChlineSaurus ChlineSaurus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fabriziofortino fabriziofortino merged commit 3604217 into apache:trunk Jan 14, 2026
1 check passed
@fabriziofortino fabriziofortino deleted the OAK-12053 branch January 14, 2026 10:11
bhabegger pushed a commit to bhabegger/jackrabbit-oak that referenced this pull request Jan 15, 2026
apache#2681)

* OAK-12053: set max analyzed offset for highlights to avoid query failures for large fields

* OAK-12053: minor improvements

* OAK-12053: minor improvements

* OAK-12053: minor improvements

* OAK-12053: minor improvements

* OAK-12053: add TODO to improve logic when upgrading to v9.x
reschke pushed a commit that referenced this pull request Feb 15, 2026
#2681)

* OAK-12053: set max analyzed offset for highlights to avoid query failures for large fields

* OAK-12053: minor improvements

* OAK-12053: minor improvements

* OAK-12053: minor improvements

* OAK-12053: minor improvements

* OAK-12053: add TODO to improve logic when upgrading to v9.x
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants