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

   ### What changes were proposed in this pull request?
   
   Apply Scalafmt to the `messageParameters` argument in 
`StructField.metadataJson`.
   This only joins the `Map` expression onto one line.
   
   ### Why are the changes needed?
   
   The formatting introduced in 8e30cf9349378248d09ac0d618c783b84d07e1b2 causes
   `dev/lint-scala` to fail with `Scalafmt: Unformatted files found` in 
`spark-sql-api`.
   The same failure appears in the CI runs for
   
[#58982](https://github.com/ruanwenjun/spark/actions/runs/35817017129/job/107041277772)
   and 
[#58984](https://github.com/wangyum/spark/actions/runs/35815752499/job/107037531964).
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Reproduced the SQL API Scalafmt failure before the fix. After the fix, 
validated
   all five modules checked by the Scalafmt portion of `dev/lint-scala`:
   
   ```sh
   mvn scalafmt:format -Dscalafmt.skip=false -Dscalafmt.validateOnly=true \
     -Dscalafmt.changedOnly=false -pl sql/api -pl sql/connect/common \
     -pl sql/connect/server -pl sql/connect/shims -pl sql/connect/client/jvm
   ```
   
   `git diff --check` also passed. No tests were added or run because this 
change
   only affects formatting.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: OpenAI Codex (GPT-6)
   


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