Skip to content

[7.x.x] Correct the data problems with Fields in the Lucene Full Text Index #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jul 19, 2025

Conversation

adamretter
Copy link
Member

@adamretter adamretter commented Jul 16, 2025

Previously the PRs eXist-db/exist#4253 and eXist-db/exist#4541 introduced some serious problems with Fields in the Lucene Full Text Index. This PR fixes the following problems:

  1. Silently swallowing value and type errors that occurred when indexing fields.
  2. Missing matches from ft:query when used on the Context Item within a predicate.
  3. Indexing fields as strings that were not valid XDM values for the explicitly indicated types.
  4. Silently dropping Timezone information when storing xs:date and xs:time values into fields.
  5. Corruption of xs:integer and xs:decimal values that exceed 32 bits.
  6. Missing support for all XDM types. Previously only 14 XDM atomic types were supported (xs:integer, xs:long, xs:unsignedLong, xs:int, xs:unsignedInt, xs:short, xs:unsignedShort, xs:decimal, xs:double, xs:float, xs:date, xs:time, xs:dateTime, xs:string). This PR adds support for all of the 45 XDM 3.1 atomic types!

Closes eXist-db/exist#5193

@adamretter adamretter added this to the elemental-7.1.1 milestone Jul 16, 2025
@adamretter adamretter self-assigned this Jul 16, 2025
@adamretter adamretter added bug Something isn't working enhancement New feature or request labels Jul 16, 2025
@adamretter adamretter force-pushed the 7.x.x/hotfix/binary-field-types branch from b8b77f6 to b365068 Compare July 16, 2025 19:50
@adamretter adamretter force-pushed the 7.x.x/hotfix/binary-field-types branch from b365068 to 5a1586b Compare July 19, 2025 14:50
@adamretter adamretter merged commit 90e8023 into develop-7.x.x Jul 19, 2025
0 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant