sarutak commented on pull request #31569: URL: https://github.com/apache/spark/pull/31569#issuecomment-783002002
I don't think replacing all the occurence of `'` with `$""` or `col()` should meet Spark 3.2.0. But should we not use `'` in the Spark code and not let end users use the syntax any more as we can't ensure that the Scala community take their decision back? Especially, should we encourage end users to use `$""` or `col()` syntax to mitigate the migration cost? So, I think we should do the following tasks first. * Replace `'` syntax with `$""` or `col()` in examples and documents. * Deprecate APIs which take `Symbol` type parameters. And the time it's necessary, shall we replacing the rest of `'`? We might need to further discuss when is the time to replacing them (judge point), and how we avoid use `'` in the Spark code any more. What do you think? ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
