fallintoplace opened a new pull request, #1740:
URL: https://github.com/apache/iceberg-go/pull/1740

   ## What changed
   
   Handle strict truncate projections at the minimum and maximum integer bounds 
without passing sentinel literals into the transform.
   
   ## Why
   
   Projecting predicates such as `id > math.MaxInt32` currently increments the 
boundary to an above-max literal. The truncate transform does not support that 
literal and panics with `invalid literal type`.
   
   The projection API cannot return an always-false unbound predicate, so this 
uses no partition projection for these cases. That is conservative and leaves 
the exact row predicate in place.
   
   ## Testing
   
   - added cases for strict predicates at both int32 bounds
   - added cases for strict predicates at both int64 bounds
   - `go test ./...`
   - `go vet ./...`


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