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();
   }
 

Reply via email to