[LUCENE-5599] HttpReplicator uses a lot of CPU for large files

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.7.1
• Fix Version/s: 4.8.1, 4.9, 6.0
• Component/s: modules/replicator
• Lucene Fields: New, Patch Available
• Assignee: Shai Erera
• Reporter: Christoph Kaser
• Created: 11/Apr/14 08:42
• Updated: 28/Aug/22 14:05
• Resolved: 16/Apr/14 14:43

Description

The method responseInputStream of HttpClientBase wraps an InputStream in order to close it when it is done reading. However, the wrapper only overwrites the single-byte read() method, every other method is delegated to its parent (java.io.InputStream). Therefore, the more efficient read-methods like read(byte[] b) are all implemented by reading one byte after the other.

In my test, it took 20 minutes to copy an index of 38 GB. With the provided small patch, this was reduced to less than 10 minutes.