Re: [PR] ci: drop non-existent suites from the Linux and macOS test matrices [datafusion-comet]

["ErikBPF (via GitHub)" <gi...@apache.org>]

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: github-unsubscribe@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org