Skip to content

chore: rename BaseInstantTime to LogFileInstantTime in log summary#17567

Merged
yihua merged 1 commit intoapache:masterfrom
shubhampatel28:shubhamp/log-file-instant-time-rename
Dec 11, 2025
Merged

chore: rename BaseInstantTime to LogFileInstantTime in log summary#17567
yihua merged 1 commit intoapache:masterfrom
shubhampatel28:shubhamp/log-file-instant-time-rename

Conversation

@shubhampatel28
Copy link
Contributor

Describe the issue this Pull Request addresses

  • Renamed BaseInstantTime to LogFileInstantTime in the summary log message for BaseHoodieLogRecordReader
  • In table version 9, the instant time extracted from log files reflects the delta commit time, not the base file instant time. The previous naming was confusing, so renamed to LogFileInstantTime for clarity.

Summary and Changelog

  • change text message for logging

Impact

None

Risk Level

None

Documentation Update

None

Contributor's checklist

  • [ x] Read through contributor's guide
  • [ x] Enough context is provided in the sections above
  • [ x] Adequate tests were added if applicable

@github-actions github-actions bot added the size:XS PR with lines of changes in <= 10 label Dec 11, 2025
@yihua yihua force-pushed the shubhamp/log-file-instant-time-rename branch from 30fa715 to 8c9b81b Compare December 11, 2025 18:28
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@yihua yihua merged commit 3c8d466 into apache:master Dec 11, 2025
70 checks passed
@yihua yihua changed the title chore: rename BaseInstantTime to LogFileInstantTime in log summary chore: [17567] rename BaseInstantTime to LogFileInstantTime in log summary Dec 12, 2025
@yihua yihua changed the title chore: [17567] rename BaseInstantTime to LogFileInstantTime in log summary [17567] chore: rename BaseInstantTime to LogFileInstantTime in log summary Dec 12, 2025
@yihua yihua changed the title [17567] chore: rename BaseInstantTime to LogFileInstantTime in log summary chore: rename BaseInstantTime to LogFileInstantTime in log summary Dec 12, 2025
@shubhampatel28 shubhampatel28 deleted the shubhamp/log-file-instant-time-rename branch February 4, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS PR with lines of changes in <= 10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants