Skip to content

Commit 7e5cbb4

Browse files
authored
GH-48444: [Python] Remove todo of implementing requested_schema in test_roundtrip_reader_capsule (#48445)
### Rationale for this change https://github.com/apache/arrow/blob/6456944f5092dedb3f80d9bc80400e857d6571c7/python/pyarrow/tests/test_cffi.py#L648 can be removed now because `requested_schema` is implemented in d6b9051fa0 and its corresponding test was added as `test_roundtrip_batch_reader_capsule_requested_schema`. ### What changes are included in this PR? This PR removes the todo of implementing requested_schema. ### Are these changes tested? I removed a comment so did not test. ### Are there any user-facing changes? No, test-only. * GitHub Issue: #48444 Authored-by: Hyukjin Kwon <[email protected]> Signed-off-by: AlenkaF <[email protected]>
1 parent 20895f7 commit 7e5cbb4

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

python/pyarrow/tests/test_cffi.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -645,7 +645,6 @@ def test_roundtrip_device_array_capsule(arr, schema_accessor, bad_type, good_typ
645645
assert schema_accessor(arr_out) == good_type
646646

647647

648-
# TODO: implement requested_schema for stream
649648
@pytest.mark.parametrize('constructor', [
650649
pa.RecordBatchReader.from_batches,
651650
# Use a lambda because we need to re-order the parameters

0 commit comments

Comments
 (0)