Skip to content

[VL] Add waitForPreloadSplitNanos in metric#11024

Merged
FelixYBW merged 1 commit intoapache:mainfrom
rui-mo:wip_preload
Nov 5, 2025
Merged

[VL] Add waitForPreloadSplitNanos in metric#11024
FelixYBW merged 1 commit intoapache:mainfrom
rui-mo:wip_preload

Conversation

@rui-mo
Copy link
Copy Markdown
Contributor

@rui-mo rui-mo commented Nov 4, 2025

What changes are proposed in this pull request?

Add waitForPreloadSplitNanos in the metric data source add split time.

dataSourceAddSplitWallNanos counts the time when the split isn't prefetched.
waitForPreloadSplitNanos counts the time when the split is prefetched but not finished when table scan read it.

Both metrics count time in table scan thread(main task thread) only.

preloadSplitPrepareTimeNanos counts the accumulated time of all prefetched split. It may count time in I/O thread

How was this patch tested?

under test

@github-actions github-actions Bot added the VELOX label Nov 4, 2025
@FelixYBW FelixYBW merged commit a318b07 into apache:main Nov 5, 2025
143 of 151 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants