Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] Allow FileIndexer to seek and skip if they need to. #3406

Merged
merged 3 commits into from
Jun 17, 2024

Conversation

leaves12138
Copy link
Contributor

Purpose

Give SeekablePositionedMappingInputStream to FileIndexer, allow them to skip to specified position if they need to.

@leaves12138 leaves12138 changed the title [core] Add SeekablePositionedMappingInputStream for file index. [core] Give SeekablePositionedMappingInputStream to FileIndexer, allow them to seek and skip if they need to. May 27, 2024
@leaves12138 leaves12138 changed the title [core] Give SeekablePositionedMappingInputStream to FileIndexer, allow them to seek and skip if they need to. [core] Give SeekablePositionedMappingInputStream to FileIndexer, allow FileIndexer to seek and skip if they need to. May 27, 2024
@leaves12138 leaves12138 changed the title [core] Give SeekablePositionedMappingInputStream to FileIndexer, allow FileIndexer to seek and skip if they need to. [core] Allow FileIndexer to seek and skip if they need to. Jun 17, 2024
@leaves12138 leaves12138 requested a review from JingsongLi June 17, 2024 03:02
@@ -29,7 +31,8 @@ public interface FileIndexer {

FileIndexWriter createWriter();

FileIndexReader createReader(byte[] serializedBytes);
FileIndexReader createReader(
SeekableInputStream inputStream, Pair<Integer, Integer> startAndLength);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

int start, int length

@JingsongLi
Copy link
Contributor

+1

@JingsongLi JingsongLi merged commit 773b7b5 into apache:master Jun 17, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants