[LUCENE-5301] All PackedInts APIs should share a common interface for random-access reads

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 4.6
• Lucene Fields: New
• Assignee: Adrien Grand
• Reporter: Adrien Grand
• Created: 23/Oct/13 15:27
• Updated: 28/Aug/22 13:55
• Resolved: 12/Nov/13 13:56

Description

It would be convenient if all PackedInts had a super-class with the long get(long index) method. Maybe this super-class could even be NumericDocValues so that doc values formats don't need to wrap eg. BlockPackedReader into this kind of construct:

        final BlockPackedReader reader = new BlockPackedReader(data, entry.packedIntsVersion, entry.blockSize, entry.count, true);
        return new LongNumericDocValues() {
          @Override
          public long get(long id) {
            return reader.get(id);
          }
        };

Instead, they could just

        return new BlockPackedReader(data, entry.packedIntsVersion, entry.blockSize, entry.count, true);