databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.85k stars 750 forks source link

chore(query): Simplified sort simple rows implementation #16382

Closed forsaken628 closed 2 months ago

forsaken628 commented 2 months ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Simplified sort simple rows implementation

Tests

Type of change


This change isโ€‚Reviewable

sundy-li commented 2 months ago

@forsaken628 does this pr save memory usage per row ?

forsaken628 commented 2 months ago

@forsaken628 does this pr save memory usage per row ?

should be minimal, SimpleRow has a very short life cycle.