fix: bypass the arrow take for struct array

[BubbleCal]

No description provided.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 45.16129% with 17 lines in your changes missing coverage. Please review.

Project coverage is 78.45%. Comparing base (33ae43b) to head (516dd19).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
rust/lance/src/dataset/take.rs	45.16%	11 Missing and 6 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3500      +/-   ##
==========================================
- Coverage   78.48%   78.45%   -0.04%     
==========================================
  Files         252      252              
  Lines       94011    94073      +62     
  Branches    94011    94073      +62     
==========================================
+ Hits        73783    73802      +19     
- Misses      17232    17270      +38     
- Partials     2996     3001       +5     

Flag	Coverage Δ
unittests	78.45% <45.16%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[westonpace]

Thanks for this workaround.

[westonpace]

Minor nit: we could probably just return StructArray here instead of Arc<dyn Array> since the next thing you do is convert it to a RecordBatch anyways.