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.

Sign up for GitHub

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

Jump to bottom

Load: Fix some issues during loading tsfiles of old version #14649

Merged
merged 14 commits into from
Jan 14, 2025
Merged

Load: Fix some issues during loading tsfiles of old version #14649

merged 14 commits into from
Jan 14, 2025

Conversation

shuwenwei
Copy link
Collaborator

@shuwenwei shuwenwei commented Jan 8, 2025 •
edited
Loading

Description

  1. ArrayDeviceTimeIndex does not update minStartTime and maxEndTime
  2. Reading old version resource file with mods file may cause error
  3. TsFile genrated by Load has wrong ModificationFile instance, which refer to the source file of Load.
  4. Old mod file cannot be read by LoadTsFileTreeSchemaCache

@SteveYurongSu SteveYurongSu self-assigned this Jan 8, 2025
@shuwenwei
ArrayDeviceTimeIndex does not update minStartTime and maxEndTime & Re…
849cd7f 
…ading old version resource file with mods file may cause error
@shuwenwei shuwenwei marked this pull request as draft January 8, 2025 04:10
@shuwenwei shuwenwei marked this pull request as ready for review January 9, 2025 10:08
@SteveYurongSu SteveYurongSu changed the title Fix some issues during loading tsfiles of old version Load: Fix some issues during loading tsfiles of old version Jan 10, 2025
@SteveYurongSu SteveYurongSu merged commit b51f63c into apache:master Jan 14, 2025
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jt2594838 jt2594838 jt2594838 approved these changes

@SteveYurongSu SteveYurongSu Awaiting requested review from SteveYurongSu

Assignees

@SteveYurongSu SteveYurongSu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shuwenwei @jt2594838 @SteveYurongSu