zanmato1984 commented on code in PR #43389:
URL: https://github.com/apache/arrow/pull/43389#discussion_r1709270268
##########
cpp/src/arrow/testing/random.h:
##########
@@ -433,12 +433,18 @@ class ARROW_TESTING_EXPORT RandomArrayGenerator {
///
/// \param[in] size the size of the array to generate
/// \param[in] byte_width the byte width of fixed-size binary items
+ /// \param[in] min_byte the lower bound of each byte in the binary
determined by the
+ /// uniform distribution
+ /// \param[in] max_byte the upper bound of each byte in the binary
determined by the
+ /// uniform distribution
/// \param[in] null_probability the probability of a value being null
/// \param[in] alignment alignment for memory allocations (in bytes)
/// \param[in] memory_pool memory pool to allocate memory from
///
/// \return a generated Array
std::shared_ptr<Array> FixedSizeBinary(int64_t size, int32_t byte_width,
+ uint8_t min_byte =
static_cast<uint8_t>('A'),
+ uint8_t max_byte =
static_cast<uint8_t>('z'),
Review Comment:
Sounds good, will do!
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
