[I] [C++] Compute: RowEncoder supports a unsafe_encoded_row to get the std::string_view for row [arrow]

[via GitHub]

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

   ### Describe the enhancement requested
   
   Currently, a RowTable always get row by `encoded_row`, which returns an 
`std::string`
   
   We could supports an unsafe_encoded_row, like below:
   
   ```c++
     /// Returns the encoded representation of the row at index i.
     ///
     /// Note: this method is unsafe because it returns a string_view that
     /// might be invalidated by subsequent calls to EncodeAndAppend or
     /// Clear.
     inline std::string_view unsafe_encoded_row(int32_t i) const {
       if (i == kRowIdForNulls()) {
         return {reinterpret_cast<const char*>(encoded_nulls_.data()),
                            encoded_nulls_.size()};
       }
       int32_t row_length = offsets_[i + 1] - offsets_[i];
       return {reinterpret_cast<const char*>(bytes_.data() + offsets_[i]),
                          static_cast<uint32_t>(row_length)};
     }
   ```
   
   ### Component(s)
   
   C++


-- 
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