bamaer commented on PR #8802:
URL: https://github.com/apache/hop/pull/8802#issuecomment-6075253891

   Thanks @kotwal-itpro, all points are resolved. The new tests are a nice 
touch, and thanks for catching the `timestamp_ns` and `next()` bugs along the 
way.
   
   Verified locally on `b8135d2490`:
   - Lakehouse tests (61) and Spark tests (179) pass in UTC. The lakehouse 
tests also pass with the JVM set to Los Angeles and Berlin. Against the pre-fix 
classes, the new tests fail as expected.
   - `hop-run` on an assembled client against an Iceberg table on an 
S3-compatible store, read through a MinIO connection:
     - With 3 copies, every row is read exactly once.
     - Time travel to an earlier snapshot returns the right rows.
     - `date` and `timestamp` keep their wall-clock values in Los Angeles and 
Berlin.
     - The pre-fix jar fails on the same table with `does not support random 
access`.
   


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