Hyukjin Kwon
Merged to master and branch-3.3.
@jyong-somnambulist please file a JIRA, and link it to the PR title. See also https://spark.apache.org/contributing.html. The codebase is written in English so let's stick to one language.
Will take a close look next Monday in KST.
My comments are just nits. I will merge this in first to move forward. Merged to master.
Looks making sense to me ... @MaxGekk @gengliangwang or @cloud-fan
Looks fine from a cursory look .. but let me add some more Python and SQL people here - @cloud-fan, @maryannxue, @viirya @ueshin @BryanCutler FYI
BTW, @LucaCanali do you mind taking a look at the test failures at https://github.com/LucaCanali/spark/runs/5059530132?
Hm, I doubt if the timeout is the actual problem. This test doesn't take too much time if you run locally actually.
@Shockang the test in the GitHub Actions fails because your fork is based on the old master branch. do you mind rebasing it to sync to the latest `master` branch?