tree 3e56fc98bc9d25296914f1fe3c38da44e0f4636f
parent 5ce35f8993151fe8cd8c4fb7865aee102752bc5b
author Fangchen Li <fangchen.li@outlook.com> 1784015721 -0700
committer GitHub <noreply@github.com> 1784015721 +0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJqVetpCRC1aQ7uu5UhlAAA4x4QABO5wshmCoQqdObBFNrgadl0
 Rw2P8XKsyP1agClhCaQRqeihALntbCDvqfOxg0/Fio+JI3v9wgHu6KYs8zQIKtRM
 vVleIlbb+5I1u1Opcywn6wlsnlzJ9PBZ4BOAJGa6l6vPfpFJbKW8IjPxCo9klERb
 MU2ApOizKHXfnzZNSzeLEsSFH0GF9Sz9xK3QVkURFhQ5oJ+12i3wyraAgA8j21pw
 sS2bJixbI3CD13QMuuIVa+qm4NVnas20dH6i/JKo4IjG1ZFGzr0+ywTZuYaKyOFA
 r2xtZgjuOqc8awzhtAl0Oe6GpsBFT9D/RFTSYV7BJ3y03XFr5OH7g8nr2MVYhhi1
 OGXXI2nRbNPNseKrpzkJO+ULWg4izwouIASZZ0x5ZRioMGSYn8MzDEuB/8JDKONM
 RwYFyubBDEdKM7jv2Gtf+M1Zr8DdmApZ2j63Yg6wbSbY8PVpngfJmgWKBOXh3ZF4
 YpjEQIy/zOxs0C5OP4mTEOeI3MiHEBxViXix66NisFWUgbS3ufsf0cuvenAlKY5y
 fxJynEH8ITTisLfvR3NHKCRMLvnHh+ScrhgaypcdPQKTxnntKOd/g3qY+rCYhxje
 Q7sIfK3NHGJYWmWqS9iYdMgN3jYJOSVtsLyZbqT9CdcFbtH5C4p28TKhui9ROtL7
 pGQr8pLu7zuuU77m8CX0
 =X01N
 -----END PGP SIGNATURE-----
 

fix: no panic on empty partition in register_record_batches (#1627)

* fix: no panic on empty partition in register_record_batches

`register_record_batches` read the schema via unchecked `partitions.0[0][0]`,
which panics when a partition contains no record batches (e.g. the batches of
an empty pyarrow table, whose `to_batches()` returns `[]`).

Take the schema from the first available batch across all partitions and return
a clear error when there is none, mirroring the empty-table handling added for
`from_arrow_table` in #613. A non-empty later partition now also works instead
of panicking on an empty leading one.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* refactor: address review feedback

- Use find_map(|p| p.first()) instead of flatten().next() to make the
  "first batch in any partition" intent clearer (per @kosiew).
- Extend regression test to also assert the empty outer partition list
  case ([]) raises the same ValueError.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>