You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Tim-Brooks (via GitHub)" <gi...@apache.org> on 2026/05/22 03:34:18 UTC
[PR] Add DictionaryColumn for SORTED / SORTED_SET dvs [lucene]
Tim-Brooks opened a new pull request, #16101:
URL: https://github.com/apache/lucene/pull/16101
Adds a new Column subtype for columnar addBatch ingestion where the
term universe is known upfront (Arrow/Parquet-style). Callers supply
a List<BytesRef> dictionary plus per-doc ordinals via OrdinalsCursor
(dense) or OrdinalsTupleCursor (sparse / multi-valued). The writer
maintains a per-batch ord→hash translation table so each distinct
dictionary entry pays one BytesRefHash probe instead of one per doc.
Wires through SortedDocValuesWriter.addOrdinalTuples /
addDenseOrdinalValues, SortedSetDocValuesWriter.addOrdinalTuples,
IndexingChain.processDictionaryColumn, ColumnValidation, and
ColumnFieldAdapter (so stored fields and term inversion work via
the existing BinaryColumnAdapter with a dictionary-resolving
cursor wrapper).
Tests in TestDictionaryColumn cover SORTED dense/sparse, SORTED_SET
(with intra-doc dedup, duplicate dict entries, multi-batch merges),
stored binary/string, inverted tokenized/untokenized fields, and
validation rejections. Two randomized cross-check tests are
included pending investigation of a divergence vs the plain-field
oracle.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [PR] Add DictionaryColumn for SORTED / SORTED_SET dvs [lucene]
Posted by "ChrisHegarty (via GitHub)" <gi...@apache.org>.
ChrisHegarty merged PR #16101:
URL: https://github.com/apache/lucene/pull/16101
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org