NVIDIA / NVIDIA/cuvs

[IMP] Lucene: initialize configured accelerated codecs once

Open
#2,616 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Cuda
Stars
854
Forks
236
Avg merge
3d 3h
Merged PRs (30d)
62

Description

Problem

Parameterized accelerated-codec constructors currently delegate through constructors that install a default vectors format, then immediately replace it with the requested format. This performs redundant initialization and can trigger avoidable side effects.

Proposed change

Have each parameterized constructor call FilterCodec directly and initialize exactly one vectors format with the requested parameters. Preserve every public constructor, codec name, delegate, default, and persisted format.

Apply this narrowly to current main; do not restore the native-buffer implementation reverted by #2597.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the parameterized accelerated-codec constructors on current main and trace their constructor delegation and vectors-format initialization. Verify that each requested format is initialized once while public constructors, codec names, delegates, defaults, and persisted formats remain unchanged; do not restore the native-buffer implementation reverted by #2597.

Written by the indexing model from the issue text.

Assessment

Domain
search
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.