[FEAT] connect: add df.limit and df.first

[andrewgazelka]

No description provided.

[andrewgazelka]

• [CHORE] connect: add tests for df.take() method #3385
• [FEAT] connect: add df.limit and df.first #3309 👈 (View in Graphite)
• [FEAT] consolidate Spark session fixture into conftest.py #3341 : 3 other dependent PRs (#3326 , #3342 , #3343 )
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Performance Report

Merging #3309 will degrade performances by 48.97%

_{Comparing andrew/connect-first (e099f2e) with main (3394a66)}

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 15 untouched benchmarks

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

Benchmarks breakdown

	Benchmark	main	andrew/connect-first	Change
⚡	test_iter_rows_first_row[100 Small Files]	324.1 ms	284 ms	+14.11%
❌	test_show[100 Small Files]	15.9 ms	31.2 ms	-48.97%

[codecov]

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 77.35%. Comparing base (3394a66) to head (e099f2e).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/daft-connect/src/translation/logical_plan.rs	87.50%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3309   +/-   ##
=======================================
  Coverage   77.35%   77.35%           
=======================================
  Files         685      685           
  Lines       83631    83639    +8     
=======================================
+ Hits        64695    64703    +8     
  Misses      18936    18936           

Files with missing lines	Coverage Δ
src/daft-connect/src/translation/logical_plan.rs	71.42% <87.50%> (+9.89%)	⬆️

... and 1 file with indirect coverage changes

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[andrewgazelka]

Merge activity

• Nov 21, 11:41 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Nov 21, 11:41 AM EST: Graphite couldn't merge this PR because it failed optional checks and "ignore optional checks" was not selected.