uros-b opened a new pull request, #57988:
URL: https://github.com/apache/spark/pull/57988

   ### What changes were proposed in this pull request?
   
   Fixes four grammar errors in build-file comments:
   
   - `sql/catalyst/pom.xml` and `sql/core/pom.xml`: `However, a closed due to 
"Cannot Reproduce" Maven bug` becomes `However, a Maven bug closed as "Cannot 
Reproduce"`.
   - `sql/core/pom.xml`: `dependencies are needed for maven test builds on 
later hadoop releases` becomes `dependencies is needed for Maven test builds on 
later Hadoop releases` (the subject is the singular "declaration"; also fixes 
proper-noun casing and a missing space before the comment close).
   - `project/SparkBuild.scala`: `There are a reasons` becomes `There are 
reasons`.
   
   ### Why are the changes needed?
   
   These comments explain non-obvious build decisions, so they are worth 
keeping readable. The first is duplicated verbatim across the two POMs, which 
is how the error propagated.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Comment text only: Maven and SBT never parse it, so there is no build 
impact, and no version, coordinate, scope, or ordering is touched. Both POMs 
were confirmed to still parse as well-formed XML.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Opus 4.8)
   


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