Narwhal-fish opened a new pull request, #859:
URL: https://github.com/apache/iceberg-go/pull/859

   ## Problem
   From #843
   `SortField.Transform` can be nil when constructing a `SortOrder`, which 
causes
   transform validation to be skipped. This allows invalid sort orders to be 
created.
   
   ## Solution
   
   Require `SortField.Transform` to be non-nil. Return `ErrInvalidTransform` 
when
   a sort field has no transform, and always perform transform compatibility 
checks.
   
   ## Test
   
   - Added test to ensure nil transform is rejected
   - Added test to verify valid transform is accepted
   - Added compatibility test with valid schema and transform


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to