KKcorps opened a new pull request, #20151: URL: https://github.com/apache/pinot/pull/20151
Configuration policy combinations and FLOAT value conversions need explicit coverage at their owning boundaries. This adds cases to three existing classes: - `TableConfigUtilsTest` covers protected/restricted consumption policies for FULL and PARTIAL upsert, out-of-order dropping, and unknown validation-skip tokens. - `RealtimeSegmentConverterTest` checks omitted, explicit non-finite and finite MV FLOAT values through mutable indexing, segment conversion and reopen, across dictionary/raw storage and row/column-major builders. - `JsonResponseEncoderTest` asserts preservation of negative infinity, positive infinity and NaN in FLOAT_ARRAY round trips, with a finite control. **Draft: the codec reproduction intentionally fails for non-finite values.** JSON represents these elements as strings, but the FLOAT_ARRAY decoder uses `JsonNode.floatValue()`, producing zero. The assertions remain enabled; the production decoder needs a separate fix before this is ready to merge. These are ordinary discovered cases in the existing classes, with no environment switch or manual selection required by the code. No production code, workflow or configuration changes are included. -- 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]
