[I] [C++][Parquet] Parquet binary length overflow exception should contain the length of binary [arrow]

[via GitHub]

mapleFU opened a new issue, #39843:
URL: https://github.com/apache/arrow/issues/39843

   ### Describe the enhancement requested
   
   ```
     template <typename ArrayType>
     void PutBinaryArray(const ArrayType& array) {
       PARQUET_THROW_NOT_OK(::arrow::VisitArraySpanInline<typename 
ArrayType::TypeClass>(
           *array.data(),
           [&](::std::string_view view) {
             if (ARROW_PREDICT_FALSE(view.size() > kMaxByteArraySize)) {
               return Status::Invalid("Parquet cannot store strings with size 
2GB or more");
             }
             PutByteArray(view.data(), static_cast<uint32_t>(view.size()));
             return Status::OK();
           },
           []() { return Status::OK(); }));
     }
   ```
   
   The code above works well, however, I think it's better to have a length for 
`view`, which would help debugging.
   
   ### Component(s)
   
   C++, Parquet


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org