panbingkun

Results 183 comments of panbingkun

Hi @HyukjinKwon, when I merge this PR, the following prompt appears: ```shell Check if the JIRA information is as expected (y/N): y JIRA is unassigned, choose assignee [0] loong (Reporter)...

Merged to master & branch-3.5 Thank you for helping with the review @HyukjinKwon, ❤️ Welcome to the Apache Spark community, [@xunxunmimi5577](https://github.com/xunxunmimi5577) ! I added you to the Apache Spark contributor...

When I was preparing to upgrade `scalafmt` from `3.8.2` to `3.8.3` in `Spark` repo, I found the above problem. The command is as follows: ``` ./build/mvn scalafmt:format -Dscalafmt.skip=false -Dscalafmt.validateOnly=false -Dscalafmt.changedOnly=false...

``` (base) ➜ spark-trunk git:(master) ✗ scalafmt --version scalafmt 3.8.3 (base) ➜ spark-trunk git:(master) ✗ scalafmt -c dev/.scalafmt.conf sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala (base) ➜ spark-trunk git:(master) ✗ git diff ```

Sorry, this is my first time submitting an issue to this repo, but it is indeed a issue. I have updated the description of this PR and it is normal...

cc @AA-Turner If you have time, can you help review this logic?

> * You need tests (autodoc + autosummary) > * You need to add a CHANGES entry. Okay, let me add it later.

> Hi @panbingkun @cloud-fan the CI is down. > > Did you forget to generate and push the HTML content? Oh, let me fix it, thanks. FOLLOWUP PR: https://github.com/apache/spark-website/pull/529