ErikBPF commented on PR #6099: URL: https://github.com/apache/datafusion-comet/pull/6099#issuecomment-5951379885
Updated against pinned main `63fd1c9e`. Rebase conflicts were resolved while retaining upstream matrix additions and `CometUnboundedShuffleMemoryAllocatorSuite`. Published ancestry is retained with an identical-tree merge, without force-pushing. The reverse suite-declaration check remains: explicitly named matrix suites must resolve to real Scala suite declarations. Stale entries were removed from both Linux and macOS matrices; `ParquetEncryptionITCase` remains covered. There are 151 matrix entries per workflow, not 151 distinct Scala suites. Verification on Apollo: - The original stale matrices produced 12 expected diagnostics; corrected matrices passed. - Deliberately invalid suite names produced the expected two diagnostics. - CI configuration, syntax, diff checks and workflow-only actionlint passed; independent source review found no introduced defect. Full actionlint still reports unchanged ShellCheck findings in unedited shell fragments (`SC2006`, `SC2086`, `SC2129`); it is not being reported as fully green. No Scala/runtime change was made in this PR. Updated head: `81a7b21b53d1be6a81244a8b8bb278d91f6e3c2a` (tested tree unchanged from `cb525947eeef4e3c13bacca9a0446a1607ef0bf4`). -- 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]
