feat(connect): add df.filter

[andrewgazelka]

No description provided.

[andrewgazelka]

• test(connect): df.where #3358
• feat(connect): add df.filter #3346 👈 (View in Graphite)
• [FEAT] connect: add binary operators #3350 : 3 other dependent PRs (#3351 , #3376 , #3387 )
• [FEAT] connect: support basic column operations #3362 : 1 other dependent PR (#3363 )
• [FEAT] connect: to_daft_* use ref instead of value #3355 : 2 other dependent PRs (#3357 , #3360 )
• [FEAT] connect: add alias support #3342 : 6 other dependent PRs (#3344 , #3345 , #3348 and 3 others)
• [FEAT] consolidate Spark session fixture into conftest.py #3341 : 3 other dependent PRs (#3309 , #3326 , #3343 )
• main

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

[codspeed-hq]

CodSpeed Performance Report

Merging #3346 will degrade performances by 52.34%

_{Comparing andrew/connect-filter (fecfc7d) with main (c3088d6)}

Summary

❌ 1 regressions
✅ 16 untouched benchmarks

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

Benchmarks breakdown

	Benchmark	main	andrew/connect-filter	Change
❌	test_iter_rows_first_row[100 Small Files]	105.9 ms	222.2 ms	-52.34%

[codecov]

Codecov Report

Attention: Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.

Project coverage is 77.67%. Comparing base (c3088d6) to head (fecfc7d).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...aft-connect/src/translation/logical_plan/filter.rs	81.81%	2 Missing ⚠️
src/daft-connect/src/translation/logical_plan.rs	66.66%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3346   +/-   ##
=======================================
  Coverage   77.67%   77.67%           
=======================================
  Files         713      714    +1     
  Lines       87116    87130   +14     
=======================================
+ Hits        67670    67681   +11     
- Misses      19446    19449    +3     

Files with missing lines	Coverage Δ
src/daft-connect/src/translation/logical_plan.rs	69.09% <66.66%> (-0.14%)	⬇️
...aft-connect/src/translation/logical_plan/filter.rs	81.81% <81.81%> (ø)

... and 3 files with indirect coverage changes

[raunakab]

Not too familiar with this codebase, but looks good to me. LGTM!

[andrewgazelka]

Merge activity

• Dec 11, 4:54 PM EST: A user merged this pull request with Graphite.