[HBASE-18825] Use HStoreFile instead of StoreFile in our own code base and remove unnecessary methods in StoreFile interface

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0-alpha-1, 2.0.0-alpha-3
• Fix Version/s: 2.0.0-alpha-4, 2.0.0
• Component/s: Coprocessors
• Hadoop Flags: Incompatible change, Reviewed

Release Note

Cleanup the StoreFile interface.

The metadata keys are moved to HStoreFile.

These methods are removed:
CacheConfig getCacheConf();
byte[] getMetadataValue(byte[] key);
boolean isCompactedAway();
boolean isReferencedInReads();
void initReader() throws IOException;
StoreFileScanner getPreadScanner(boolean cacheBlocks, long readPt, long scannerOrder, boolean canOptimizeForNonNullColumn);
StoreFileScanner getStreamScanner(boolean canUseDropBehind, boolean cacheBlocks, boolean isCompaction, long readPt, long scannerOrder, boolean canOptimizeForNonNullColumn) throws IOException;
StoreFileReader getReader();
void closeReader(boolean evictOnClose) throws IOException;
void markCompactedAway();
void deleteReader() throws IOException;

Notice that these methods are still available in HStoreFile.

And the return value of getFirstKey and getLastKey are changed from Cell to Optional<Cell> to better indicate that they may not be available.

• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 15/Sep/17 08:58
• Updated: 17/Jun/22 05:22
• Resolved: 25/Sep/17 03:00

Description

Use generic types to avoid too many casts.

Issue Links

• links to: Review Board