Fokko opened a new pull request, #6140:
URL: https://github.com/apache/iceberg/pull/6140

   Instead of just supplying the unbound expression to the evaluator directly, 
we created a bogus one and replaced the bound expression with the one we wanted 
to test. But introduced a bug in the test because all the strings and doubles 
were converted to a long because that was the schema that was used to 
initialize the evaluator.
   
   This PR changes the tests to supply the correct schema, and includes the 
binding in the tests as well


-- 
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...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to