This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new e21d057a22 GH-48880: [Ruby] Fix a bug that Arrow::ExecutePlan nodes
may be GC-ed (#48919)
e21d057a22 is described below
commit e21d057a225396f21091f00c8743b90057bdb89f
Author: Sutou Kouhei <[email protected]>
AuthorDate: Mon Jan 26 11:02:42 2026 +0900
GH-48880: [Ruby] Fix a bug that Arrow::ExecutePlan nodes may be GC-ed
(#48919)
### Rationale for this change
We must mark all nodes in `Arrow::ExecutePlan` but only the first node is
marked.
### What changes are included in this PR?
Fix typos in variable name.
### Are these changes tested?
Yes.
### Are there any user-facing changes?
Yes.
* GitHub Issue: #48880
Authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
---
ruby/red-arrow/ext/arrow/arrow.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ruby/red-arrow/ext/arrow/arrow.cpp
b/ruby/red-arrow/ext/arrow/arrow.cpp
index 404ec8996f..0c582d0707 100644
--- a/ruby/red-arrow/ext/arrow/arrow.cpp
+++ b/ruby/red-arrow/ext/arrow/arrow.cpp
@@ -59,7 +59,7 @@ namespace red_arrow {
{
auto plan = GARROW_EXECUTE_PLAN(object);
auto nodes = garrow_execute_plan_get_nodes(plan);
- for (auto node = nodes; nodes; nodes = g_list_next(nodes)) {
+ for (auto node = nodes; node; node = g_list_next(node)) {
rbgobj_gc_mark_instance(node->data);
}
}