JingsongLi commented on PR #10758: URL: https://github.com/apache/paimon/pull/10758#issuecomment-6360121391
**[P2] Update the Ray import test for the ARRAY file layout** The tensor mapping change removes separate vector files, but [`multimodal_lerobot_ray_test.py:405`](https://github.com/apache/paimon/blob/d8709e2395500a47aca8cf5ae80ad140c8968137/paimon-python/pypaimon/tests/multimodal_lerobot_ray_test.py#L405) still asserts that the prepared files contain a `.vector.` file. Consequently, `LeRobotRayImportTest.test_writer_close_failure_aborts_prepared_files` fails on this PR: the captured files contain only `.parquet` and `.blob` files. I ran this test against the exact base (`7ee84436af`) and head (`d8709e2395`): the base passes and the head fails at this assertion. Please update the Ray test to expect the new file layout, while retaining the assertions that no snapshot is published and all prepared files are removed. ```shell python -m pytest pypaimon/tests/multimodal_lerobot_ray_test.py::LeRobotRayImportTest::test_writer_close_failure_aborts_prepared_files -q ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
