[arrow] branch main updated: GH-35538: [C++] Remove unnecessary status.h include from protobuf (#35673)

[we...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

westonpace 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 b80c655157 GH-35538: [C++] Remove unnecessary status.h include from protobuf (#35673)
b80c655157 is described below

commit b80c6551570b8324298a12dc36be1b5eba76e277
Author: Felipe Oliveira Carvalho <fe...@gmail.com>
AuthorDate: Thu May 18 17:20:53 2023 -0300

    GH-35538: [C++] Remove unnecessary status.h include from protobuf (#35673)
    
    ### Rationale for this change
    
    Newer versions of protobuf use `absl::Status` instead of `google::protobuf::util::Status`. The status variables in this file are type-annotated with `auto` and we can count on the header that declares the status-returning function we are calling to have included the header that defines `Status`.
    
    ### What changes are included in this PR?
    
    Removing an include of a protobuf header that don't exist in later versions of the library.
    
    ### Are these changes tested?
    
    I haven't tried building with protobuf 23, but I can be sure this specifically fixes the problem reported in #35538.
    
    Authored-by: Felipe Oliveira Carvalho <fe...@gmail.com>
    Signed-off-by: Weston Pace <we...@gmail.com>
---
 cpp/src/arrow/engine/substrait/serde.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cpp/src/arrow/engine/substrait/serde.cc b/cpp/src/arrow/engine/substrait/serde.cc
index a81e488bf9..b5effd7852 100644
--- a/cpp/src/arrow/engine/substrait/serde.cc
+++ b/cpp/src/arrow/engine/substrait/serde.cc
@@ -24,7 +24,6 @@
 #include <google/protobuf/descriptor.h>
 #include <google/protobuf/io/zero_copy_stream_impl_lite.h>
 #include <google/protobuf/message.h>
-#include <google/protobuf/stubs/status.h>
 #include <google/protobuf/util/json_util.h>
 #include <google/protobuf/util/message_differencer.h>
 #include <google/protobuf/util/type_resolver.h>