Skip to content

Conversation

@duanmeng
Copy link
Collaborator

No description provided.

@duanmeng duanmeng requested a review from xiaoxmeng December 24, 2025 17:14
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 24, 2025
@netlify
Copy link

netlify bot commented Dec 24, 2025

Deploy Preview for meta-velox ready!

Name Link
🔨 Latest commit e4f0037
🔍 Latest deploy log https://app.netlify.com/projects/meta-velox/deploys/694ca4fe95f7070008c36212
😎 Deploy Preview https://deploy-preview-15856--meta-velox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@duanmeng duanmeng changed the title doc: Add sort analyze blog docs: Add sort analyze blog Dec 24, 2025
@duanmeng duanmeng requested a review from pedroerp December 24, 2025 17:26
@duanmeng duanmeng force-pushed the doc branch 2 times, most recently from 9469f8b to 3182855 Compare December 24, 2025 17:34
Copy link
Contributor

@xiaoxmeng xiaoxmeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@duanmeng thanks!

for spilling.

### Non-Materialized Sort
We have implemented a [non-materializing sort strategy](https://github.com/facebookincubator/velox/pull/15157) designed to improve sorting performance. The approach materializes only the sort key columns and their original vector indices, which are then used to gather the corresponding rows from the original input vectors into the output vector after the sort is complete. It changes the `SortBuffer` to `NonMaterizedSortBuffer`, which consists of three stages:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/is complete/completes/

@meta-codesync
Copy link

meta-codesync bot commented Dec 24, 2025

@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this in D89772026.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants