[FEAT] Ellipsize glob scan paths

[anmolsingh20]

Resolves #2709

• Previously if there were multiple urls in the logical plan, it cluttered the output of 'df.explain' with massive text. Now we add ellipses if there are more than six, improving readability.
• The current test emulates multiple urls from the same test fixture 'mvp.parquet'. Ideally there should be a test fixture with multiple parts of a parquet file.

[codspeed-hq]

CodSpeed Performance Report

Merging #2809 will degrade performances by 56.78%

_{Comparing anmolsingh20:ellipsize_glob_paths_2709 (deb577f) with main (58d1856)}

Summary

❌ 2 regressions
✅ 14 untouched benchmarks

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

Benchmarks breakdown

	Benchmark	main	anmolsingh20:ellipsize_glob_paths_2709	Change
❌	test_explain[100 Small Files]	36.8 ms	43.5 ms	-15.27%
❌	test_show[100 Small Files]	49.6 ms	114.7 ms	-56.78%

[codecov]

Codecov Report

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

Please upload report for BASE (main@7fe3dbc). Learn more about missing BASE report.
Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
src/daft-scan/src/glob.rs	91.66%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2809   +/-   ##
=======================================
  Coverage        ?   63.29%           
=======================================
  Files           ?     1007           
  Lines           ?   114181           
  Branches        ?        0           
=======================================
  Hits            ?    72276           
  Misses          ?    41905           
  Partials        ?        0           

Files with missing lines	Coverage Δ
src/daft-scan/src/lib.rs	62.01% <100.00%> (ø)
src/daft-scan/src/glob.rs	87.35% <91.66%> (ø)

[samster25]

Welcome and thanks for the contribution! Just a minor fix and we should be good to go!

[samster25]

This should be 7, otherwise we take the same amount of lines and omit 1 file.

[anmolsingh20]

done

[samster25]

commented out line.

[anmolsingh20]

removed the commented line.

On a side note, I am not sure of the failing benchmarking tests. The numbers also seem highly varying, though there was no change in functionality.

[universalmind303]

@anmolsingh20 the benchmarks are a known issue right now, you can just ignore them for the time being.