[MINOR][VL] Fix unused variable when GPU was not enabled#11067
Merged
jinchengchenghh merged 1 commit intoapache:mainfrom Nov 11, 2025
Merged
[MINOR][VL] Fix unused variable when GPU was not enabled#11067jinchengchenghh merged 1 commit intoapache:mainfrom
jinchengchenghh merged 1 commit intoapache:mainfrom
Conversation
| const auto& partitionColumns = scanInfo->partitionColumns; | ||
| const auto& metadataColumns = scanInfo->metadataColumns; | ||
| #ifdef GLUTEN_ENABLE_GPU | ||
| // Under the pre-condition that all the split infos has same partition column and format. |
Contributor
There was a problem hiding this comment.
Member
Author
There was a problem hiding this comment.
But it is only used when GLUTEN_ENABLE_GPU is defined.
jinchengchenghh
approved these changes
Nov 11, 2025
GlutenPerfBot
pushed a commit
to GlutenPerfBot/gluten
that referenced
this pull request
Nov 26, 2025
Upstream Velox's New Commits: 2a50cd131 by Yolande Yan, feat(fuzzer): Add type transform for SetDigest (#15610) 1777c7523 by Zac Wen, fix: Avoid TSAN data race during cache entry initialization (#15623) 7ddcac07d by Wei He, feat(vector): Add retainedSize(totalStringBufferSize) API to return total size of all string buffers (#15578) 13ac97fe4 by Peter Enescu, fix: Remove special form SWITCH from expression fuzzer executions (#15602) d1ef88cf0 by Patrick Sullivan, fix: Improve serde behavior for zero-area polygons (#15558) 7a1ff8960 by NEUpanning, perf: Optimize copying distinct spilling vector to output vector in HashAggragtion (#15583) 4f6a521f7 by Rui Mo, feat: Support scan filter for ORC decimal reader (apache#11067) a2c77748e by NEUpanning, refactor: Remove redundant resizing of child vectors (#15600) 307e47ef4 by Pramod Satya, refactor: Remove enableConstantFolding flag from ExprCompiler (#15541) 3e94db0d5 by Eric Jia, fix: Update TableEvolutionFuzzer aggregation excluding new INTERVAL types (#15604) Signed-off-by: glutenperfbot <glutenperfbot@glutenproject-internal.com>
FelixYBW
pushed a commit
that referenced
this pull request
Nov 26, 2025
Upstream Velox's New Commits: 2a50cd131 by Yolande Yan, feat(fuzzer): Add type transform for SetDigest (#15610) 1777c7523 by Zac Wen, fix: Avoid TSAN data race during cache entry initialization (#15623) 7ddcac07d by Wei He, feat(vector): Add retainedSize(totalStringBufferSize) API to return total size of all string buffers (#15578) 13ac97fe4 by Peter Enescu, fix: Remove special form SWITCH from expression fuzzer executions (#15602) d1ef88cf0 by Patrick Sullivan, fix: Improve serde behavior for zero-area polygons (#15558) 7a1ff8960 by NEUpanning, perf: Optimize copying distinct spilling vector to output vector in HashAggragtion (#15583) 4f6a521f7 by Rui Mo, feat: Support scan filter for ORC decimal reader (#11067) a2c77748e by NEUpanning, refactor: Remove redundant resizing of child vectors (#15600) 307e47ef4 by Pramod Satya, refactor: Remove enableConstantFolding flag from ExprCompiler (#15541) 3e94db0d5 by Eric Jia, fix: Update TableEvolutionFuzzer aggregation excluding new INTERVAL types (#15604) Signed-off-by: glutenperfbot <glutenperfbot@glutenproject-internal.com> Co-authored-by: glutenperfbot <glutenperfbot@glutenproject-internal.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes are proposed in this pull request?
Fix unused variable when GPU was not enabled
How was this patch tested?
minor fix