apache / apache/lucene

deprecate either IndexInput.copyBytes or DataOutput.copyBytes [LUCENE-3657]

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

Description

Currently we have:
- IndexInput: public void copyBytes(IndexOutput out, long numBytes)
- DataOutput: public void copyBytes(DataInput input, long numBytes)

I think this is confusing, which one gets used when?!

---
Migrated from [LUCENE-3657](https://issues.apache.org/jira/browse/LUCENE-3657) by Robert Muir (@rmuir)

Contributor guide

Open the contributing guide

Research direction

Start by tracing call sites of IndexInput.copyBytes and DataOutput.copyBytes and compare the direction and semantics of each use. Review the surrounding Java APIs and any relevant tests or documentation before proposing which method should be deprecated. Done means the choice is agreed, the deprecated API and affected references are updated consistently, and the relevant tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
search
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.