LuciferYang opened a new pull request #32710:
URL: https://github.com/apache/spark/pull/32710
### What changes were proposed in this pull request?
There are several pr to fix compilation warnings related to `procedure
syntax` like SPARK-29291, SPARK-33352 and SPARK-35526, in order to prevent the
recurrence of similar problems, this pr add a compile arg to convert `procedure
syntax` related compilation warnings to compilation errors in Scala 2.13.
### Why are the changes needed?
Prevent the recurrence of compilation warnings related to `procedure syntax
is deprecated`
### Does this PR introduce _any_ user-facing change?
`procedure syntax` is no longer allowed in Spark code with Scala 2.13, for
constructors methods definition should be `this(...) = { }` not `this(...) {
}`, for without `return type` methods definition should be `def
methodName(...): Unit = {}` not `def methodName(...) {}`.
### How was this patch tested?
- Pass the GitHub Action Scala 2.13 job
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]