apache / apache/lucene

remove DocConsumer.doAfterFlush [LUCENE-5027]

Open
#6,091 3 comments 0 reactions 0 assignees View on GitHub
legacy-jira-priority:Major type:enhancement
Dominant language
Java
Stars
3.6k
Forks
1.4k
Avg merge
2d 11h
Merged PRs (30d)
88

Description

I think this stuff is dead code and confusing: talking about clearing state across segments when the DWPT is not reused?

---
Migrated from [LUCENE-5027](https://issues.apache.org/jira/browse/LUCENE-5027) by Robert Muir (@rmuir)
Attachments: [LUCENE-5027.patch](https://apache.github.io/lucene-jira-archive/attachments/LUCENE-5027/LUCENE-5027.patch) (versions: 2)

Contributor guide

Open the contributing guide

Research direction

Start by searching the Lucene source for DocConsumer.doAfterFlush and read its callers and surrounding state-management code. Review the attached LUCENE-5027.patch for context; done means the dead method and any required references are removed without leaving compilation or test failures.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
search
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.