Skip to content

feat(file_index): add file index format read and write#63

Merged
Aitozi merged 4 commits into
apache:mainfrom
devillove084:feat/impl_file_index_format
Sep 2, 2024
Merged

feat(file_index): add file index format read and write#63
Aitozi merged 4 commits into
apache:mainfrom
devillove084:feat/impl_file_index_format

Conversation

@devillove084

Copy link
Copy Markdown
Contributor

part #34

@devillove084 devillove084 force-pushed the feat/impl_file_index_format branch from 980dd62 to 96cdef9 Compare August 23, 2024 17:00
@devillove084

Copy link
Copy Markdown
Contributor Author

This update involves the reconstruction of FileIndexFormat's reading and writing functionalities based on the new FileIO. To keep the changes manageable, I’ve split the PR into smaller parts. If you have a moment, I would greatly appreciate reviews. @Xuanwo @SteNicholas

Comment thread crates/paimon/src/file_index/file_index_format.rs
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/src/file_index/file_index_format.rs Outdated
Comment thread crates/paimon/Cargo.toml Outdated
@devillove084 devillove084 force-pushed the feat/impl_file_index_format branch 2 times, most recently from 82f47ea to 5f4f2f8 Compare August 29, 2024 18:08
@devillove084

Copy link
Copy Markdown
Contributor Author

Sorry for being late again. I have corrected all your comments. Please review them when you have time to see if there are any problems. Thank you very much. @Xuanwo

Comment thread crates/paimon/src/file_index/file_index_format.rs

@Xuanwo Xuanwo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, let's move!

@devillove084 devillove084 force-pushed the feat/impl_file_index_format branch from 5f4f2f8 to c761c0e Compare September 1, 2024 15:35

@Aitozi Aitozi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@Aitozi Aitozi merged commit 65eb13a into apache:main Sep 2, 2024
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.

4 participants