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

   ### What changes were proposed in this pull request?
   
   Fixes four usage strings that disagree with the code around them:
   
   - `DirectKafkaWordCount` and `DirectKerberizedKafkaWordCount`: the class 
comment gains the missing `<groupId>` argument
   - `SkewedGroupByTest`: `Usage: GroupByTest` becomes `Usage: 
SkewedGroupByTest`
   - `JavaRecoverableNetworkWordCount`: `You arguments were` becomes `Your 
arguments were`
   
   ### Why are the changes needed?
   
   Each defect is contradicted by its own file:
   
   - Both Kafka examples gate on `args.length < 3`, destructure `val 
Array(brokers, groupId, topics) = args`, already print the three-argument form 
at runtime, and already describe `<groupId>` on the line below the usage 
string. SPARK-27973 corrected the runtime message for the first example but 
left the comment out of sync. The Java twins (`JavaDirectKafkaWordCount`, 
`JavaDirectKerberizedKafkaWordCount`) are already correct, so this closes the 
gap.
   - `SkewedGroupByTest`'s comment names `GroupByTest`, which is a different 
example that also exists, so copying the line runs the wrong job. The sibling 
`SimpleSkewedGroupByTest` self-names correctly.
   - The Scala counterpart `RecoverableNetworkWordCount` already reads `Your 
arguments were`, so the fix aligns the twins.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, other than the corrected example text.
   
   ### How was this patch tested?
   
   Comment and message text only. Each correction was cross-checked against the 
example's own argument gate, destructuring, and runtime message. A sweep of 
every Scala and Java example for a mismatch between the comment's `Usage:` 
class name and the enclosing object name found only the `SkewedGroupByTest` 
case fixed here.
   
   ### 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