LuciferYang commented on PR #43950: URL: https://github.com/apache/spark/pull/43950#issuecomment-1822092479
cc @srowen @HyukjinKwon @dongjoon-hyun Another option is to change this compilation rule to ``` -Wconf:msg=^(?=.*?method|value|type|object|trait|inheritance)(?=.*?deprecated)(?=.*?since 2.13).+$:e ``` which will convert the corresponding compilation `warning` into a compilation `error`, and Scala 2.13 deprecated APIs will be prohibited. This can save the trouble of repeated clean-up. Do you think this is a better approach, or the current PR approach is better? -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
