This is an automated email from the ASF dual-hosted git repository.
raulcd 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 ea3b8c6df0 GH-47781: [C++] Cleaned up type-limit warning in
sink_node.cc (#47782)
ea3b8c6df0 is described below
commit ea3b8c6df0df7ec9f86ca96a2525792ca76b59e3
Author: William Ayd <[email protected]>
AuthorDate: Mon Oct 13 03:23:22 2025 -0400
GH-47781: [C++] Cleaned up type-limit warning in sink_node.cc (#47782)
### Rationale for this change
This cleans up a dead code branch the compiler warns about
### What changes are included in this PR?
Removes a code branch that will never be hit
### Are these changes tested?
Yes
### Are there any user-facing changes?
No
* GitHub Issue: #47781
Authored-by: Will Ayd <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
---
cpp/src/arrow/acero/sink_node.cc | 5 -----
1 file changed, 5 deletions(-)
diff --git a/cpp/src/arrow/acero/sink_node.cc b/cpp/src/arrow/acero/sink_node.cc
index 0efb365a51..2f49822182 100644
--- a/cpp/src/arrow/acero/sink_node.cc
+++ b/cpp/src/arrow/acero/sink_node.cc
@@ -267,11 +267,6 @@ class SinkNode : public ExecNode,
return Status::Invalid(
"`backpressure::pause_if_above` must be >=
`backpressure::resume_if_below");
}
- if (sink_options.backpressure.resume_if_below < 0) {
- return Status::Invalid(
- "`backpressure::pause_if_above and backpressure::resume_if_below
must be >= 0. "
- " Set to 0 to disable backpressure.");
- }
return Status::OK();
}