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

["mapleFU (via GitHub)" <gi...@apache.org>]

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-unsubscribe@arrow.apache.org.apache.org

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

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

["mapleFU (via GitHub)" <gi...@apache.org>]

mapleFU closed issue #39843: [C++][Parquet] Parquet binary length overflow exception should contain the length of binary
URL: https://github.com/apache/arrow/issues/39843


-- 
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-unsubscribe@arrow.apache.org

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