LuciferYang opened a new pull request, #56405:
URL: https://github.com/apache/spark/pull/56405

   ### What changes were proposed in this pull request?
   
   This PR marks `pipelines` as depending on the `sql` module in Spark's test 
module graph, and updates the existing doctest expectations for the affected 
module closure.
   
   ### Why are the changes needed?
   
   The `pipelines` project depends on Spark SQL, but the test module graph 
treated it as independent. As a result, changes in SQL-related modules such as 
`sql`, `catalyst`, and `sql-api` did not select `pipelines/test` in the 
affected-module CI path.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   ```bash
   python3 -m py_compile dev/sparktestsupport/modules.py 
dev/sparktestsupport/utils.py
   PYTHONPATH=dev python3 -m doctest dev/sparktestsupport/utils.py
   git diff --check
   ```
   
   Also verified locally that changes in `core`, `sql-api`, `catalyst`, `sql`, 
and `sql/pipelines` include the `pipelines` module in the affected-module 
selection.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: OpenAI Codex
   


-- 
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]

Reply via email to