feat(connect): distinct + sort

[universalmind303]

depends on #3666

see here for proper diff

universalmind303/Daft@error-messages...connect_distinct

[codspeed-hq]

CodSpeed Performance Report

Merging #3677 will degrade performances by 32.02%

_{Comparing universalmind303:connect_distinct (a189caf) with main (809e411)}

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 25 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

	Benchmark	main	universalmind303:connect_distinct	Change
⚡	test_iter_rows_first_row[100 Small Files]	256.6 ms	218.6 ms	+17.41%
❌	test_show[100 Small Files]	16.3 ms	24 ms	-32.02%

[codecov]

Codecov Report

Attention: Patch coverage is 53.22581% with 29 lines in your changes missing coverage. Please review.

Project coverage is 75.92%. Comparing base (809e411) to head (a189caf).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
src/daft-connect/src/translation/logical_plan.rs	51.72%	28 Missing ⚠️
...onnect/src/translation/expr/unresolved_function.rs	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3677      +/-   ##
==========================================
- Coverage   77.85%   75.92%   -1.94%     
==========================================
  Files         729      728       -1     
  Lines       89983    91440    +1457     
==========================================
- Hits        70057    69425     -632     
- Misses      19926    22015    +2089     

Files with missing lines	Coverage Δ
...onnect/src/translation/expr/unresolved_function.rs	83.15% <75.00%> (+8.96%)	⬆️
src/daft-connect/src/translation/logical_plan.rs	37.02% <51.72%> (+4.81%)	⬆️

... and 20 files with indirect coverage changes

[universalmind303]

in spark, df.count() is processed as count(1), which is equivalent to our count(*)

[universalmind303]

spark doesn't support u64's so we need to cast it to i64

[kevinzwang]

LGTM. Could we add a test for count(*) as well?

[universalmind303]

LGTM. Could we add a test for count(*) as well?

AFAIK, you can't do df.count("*") in spark.