ACTION NEEDED
Lance follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

For details on the error please inspect the "PR Title Check" action.

I am trying to upgrade and received the same error in #4593. I will wait for this then 😆

Digging into this for this and also #4593 the problem I'm seeing in lance_encoding::encodings::logical::struct::tests::test_empty_struct is that the update of arrow from 55.1 to 55.2 now has a check for struct arrays having at least one field during the call to concat_structs. In arrow 56, this is corrected because it calls StructArray::try_new_with_length instead of StructArray::try_new.

I don't think we want to bump arrow to 56 with datafusion at 49 because datafusion is at arrow 55 and that would have duplicate dependencies.

I added a TODO in the code I pushed to work around this error.

Codecov Report

❌ Patch coverage is 84.44444% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.56%. Comparing base (da99228) to head (242fd2d).

@@            Coverage Diff             @@
##             main    #4598      +/-   ##
==========================================
- Coverage   80.58%   80.56%   -0.02%     
==========================================
  Files         317      317              
  Lines      119272   119281       +9     
  Branches   119272   119281       +9     
==========================================
- Hits        96111    96103       -8     
- Misses      19713    19727      +14     
- Partials     3448     3451       +3     

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

@westonpace I'm not familiar with the CI failure I'm seeing here - it looks like something related to file system errors on windows. Is that something you've seen in this repo?

I'm also not wild about the fix I put in for the test I describe in the comments above, but I believe it can be removed in the next version, DF50 which should bring Arrow 56

It looks like the CI failure I was getting earlier was resolved after rebase. I think this is ready to go.