real-mj-song commented on PR #18632: URL: https://github.com/apache/pinot/pull/18632#issuecomment-4577854408
The single CI failure (Pinot Integration Test Set 1) is unrelated to this PR's diff. Failing case: `MultiNodesOfflineClusterIntegrationTest.testExplainPlanQueryV2` — asserts on EXPLAIN PLAN output for a multi-stage query. This PR touches only two files in `pinot-plugins/pinot-input-format/pinot-arrow`; nothing in query planning, multi-stage engine, or integration tests. Assertion diff: - Expected: `Rule: AggregateProjectPullUpConstants -> Time:*` - Actual: `Rule: AggregateProjectPullUpConstants -> Time:*E-4` The `*` is a literal placeholder that the test substitutes from real timings. The actual rule execution time rendered in scientific notation (sub-millisecond, e.g. `1.5E-4ms`), and the substitution regex doesn't cover scientific notation — so it consumed only the mantissa and left `E-4` behind. Timing-dependent. Recent `Pinot Integration Tests` runs on master are passing, consistent with a timing-conditioned flake rather than a deterministic regression. -- 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]
