jduo opened a new pull request, #37942:
URL: https://github.com/apache/arrow/pull/37942

   ### Rationale for this change
   Make vector validation code more consistent with C++. Add missing checks and 
have the entry point
   be the same so that the code is easier to read/write when working with both 
languages.
   
   ### What changes are included in this PR?
   Make vector validation more consistent with Array::Validate() in C++:
   * Add validate() and validateFull() instance methods to vectors.
   * Validate that VarCharVector and LargeVarCharVector contents are valid 
UTF-8.
   * Validate that DecimalVector and Decimal256Vector contents fit within the 
supplied precision and scale.
   * Validate that NullVectors contain only nulls.
   * Validate that FixedSizeBinaryVector values have the correct length.
   
   ### Are these changes tested?
   Yes.
   
   ### Are there any user-facing changes?
   No.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to