Skip to content

Commit 25af446

Browse files
committed
test: add tests for RecordBatchStream resource cleanup
1 parent 3c47ec3 commit 25af446

File tree

1 file changed

+36
-0
lines changed

1 file changed

+36
-0
lines changed
Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
# Licensed to the Apache Software Foundation (ASF) under one
2+
# or more contributor license agreements. See the NOTICE file
3+
# distributed with this work for additional information
4+
# regarding copyright ownership. The ASF licenses this file
5+
# to you under the Apache License, Version 2.0 (the
6+
# "License"); you may not use this file except in compliance
7+
# with the License. You may obtain a copy of the License at
8+
#
9+
# http://www.apache.org/licenses/LICENSE-2.0
10+
#
11+
# Unless required by applicable law or agreed to in writing,
12+
# software distributed under the License is distributed on an
13+
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
14+
# KIND, either express or implied. See the License for the
15+
# specific language governing permissions and limitations
16+
# under the License.
17+
18+
"""Tests for RecordBatchStream resource cleanup."""
19+
20+
21+
def test_record_batch_stream_drop(ctx):
22+
"""Dropping a RecordBatchStream without iterating should release its
23+
resources so the Python interpreter can exit cleanly."""
24+
25+
stream = ctx.sql("SELECT 1").execute_stream()
26+
del stream
27+
28+
29+
def test_record_batch_stream_close(ctx):
30+
"""Explicitly closing a RecordBatchStream before dropping it is
31+
idempotent and should not raise."""
32+
33+
stream = ctx.sql("SELECT 1").execute_stream()
34+
stream.close()
35+
stream.close()
36+
del stream

0 commit comments

Comments
 (0)