vbhanuchander-lang commented on PR #17603: URL: https://github.com/apache/iceberg/pull/17603#issuecomment-5274290087
Gentle bump — this one is waiting on two things a committer can unblock. The workflow runs are all sitting at `action_required`, so CI has never executed on this branch. Could someone approve the run? Locally the new `TestUUIDUtil` cases pass, including a 1000-random-UUID equivalence check against `UUID.fromString`, and the Spark modules build on v3.5, v4.0 and v4.1. cc @nastra (you have the most history on `UUIDUtil`) and @manuzhang (you last touched `SparkValueWriters` on v4.1) — the change resolves the existing `// TODO: direct conversion from string to byte buffer` in `SparkValueWriters.UUIDWriter` by parsing the 36-byte UUID text straight into the reusable `ByteBuffer`, instead of allocating a `UUID` per value. Happy to split the API and Spark parts if you would rather review them separately. -- 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]
