xiangfu0 commented on PR #19795: URL: https://github.com/apache/pinot/pull/19795#issuecomment-6069321460
The previous [Unit Test Set 1 failure](https://github.com/apache/pinot/actions/runs/37828547907/job/113487615260) was a pre-existing Groovy fixture mismatch: expected `-0.0`, while ingestion normalizes it to `0.0`. The relevant source is unchanged between that head and current master. The separate fix is [#19803](https://github.com/apache/pinot/pull/19803). With seed `41012168809809`, the original Groovy class fails one of 24 tests; the normalized fixture passes all 24. This keeps the unrelated CI fix separate from the digest migration. All 15 current inline review threads are addressed in [4818fb8](https://github.com/apache/pinot/commit/4818fb8fbc79d81de17ce0c5bd0070eb97f300df): 505 targeted tests, 322 compatibility cases, 76 real-reader cases, and all 270 per-case legacy accuracy envelopes pass. Fresh CI is running on this exact head. -- 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]
