(arrow) branch main updated: GH-46070: [C++] Remove duplicate storage_type in JsonExtension (#46071)

[kou]

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 3961db2019 GH-46070: [C++] Remove duplicate storage_type in 
JsonExtension (#46071)
3961db2019 is described below

commit 3961db20194ecc88b5bcb9cfef4154b95450a9d5
Author: mwish <maplewish...@gmail.com>
AuthorDate: Wed Apr 9 15:17:20 2025 +0800

    GH-46070: [C++] Remove duplicate storage_type in JsonExtension (#46071)
    
    ### Rationale for this change
    
    Remove duplicate storage_type in JsonExtension
    
    ### What changes are included in this PR?
    
    Remove duplicate storage_type in JsonExtension
    
    ### Are these changes tested?
    
    Covered by existing
    
    ### Are there any user-facing changes?
    
    no
    
    * GitHub Issue: #46070
    
    Authored-by: mwish <maplewish...@gmail.com>
    Signed-off-by: Sutou Kouhei <k...@clear-code.com>
---
 cpp/src/arrow/extension/json.h | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/cpp/src/arrow/extension/json.h b/cpp/src/arrow/extension/json.h
index 89976c8073..589b341fa6 100644
--- a/cpp/src/arrow/extension/json.h
+++ b/cpp/src/arrow/extension/json.h
@@ -31,7 +31,7 @@ namespace arrow::extension {
 class ARROW_EXPORT JsonExtensionType : public ExtensionType {
  public:
   explicit JsonExtensionType(const std::shared_ptr<DataType>& storage_type)
-      : ExtensionType(storage_type), storage_type_(storage_type) {}
+      : ExtensionType(storage_type) {}
 
   std::string extension_name() const override { return "arrow.json"; }
 
@@ -48,9 +48,6 @@ class ARROW_EXPORT JsonExtensionType : public ExtensionType {
   static Result<std::shared_ptr<DataType>> Make(std::shared_ptr<DataType> 
storage_type);
 
   static bool IsSupportedStorageType(Type::type type_id);
-
- private:
-  std::shared_ptr<DataType> storage_type_;
 };
 
 /// \brief Return a JsonExtensionType instance.