Skip to content

Commit

Permalink
[SPARK-44239][SQL][FOLLOWUP] Do not disable vector memory optimizatio…
Browse files Browse the repository at this point in the history
…n when hugeVectorThreshold=0 to align its document

### What changes were proposed in this pull request?

Minor fix: Update default hugeVectorThreshold from 0 to -1 in code to match the documentation.
Keep consistent with the documentation

### Why are the changes needed?

Keep code and documentation consistent

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Exists UT

### Was this patch authored or co-authored using generative AI tooling?

No

Closes #47988 from wankunde/minor_fix.

Authored-by: wankunde <[email protected]>
Signed-off-by: Kent Yao <[email protected]>
  • Loading branch information
wankunde authored and yaooqinn committed Sep 6, 2024
1 parent 9269a0b commit e9a6230
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ public void reset() {
numNulls = 0;
}

if (hugeVectorThreshold > 0 && capacity > hugeVectorThreshold) {
if (hugeVectorThreshold > -1 && capacity > hugeVectorThreshold) {
capacity = defaultCapacity;
releaseMemory();
reserveInternal(capacity);
Expand Down

0 comments on commit e9a6230

Please sign in to comment.