[GitHub] [spark] mallman removed a comment on pull request #31569: [SPARK-34443][CORE] Replace symbol literals with Symbol constructor invocations to comply with Scala 2.13

[GitBox <gi...@apache.org>]

mallman removed a comment on pull request #31569:
URL: https://github.com/apache/spark/pull/31569#issuecomment-780028621


   I want to offer a rant on the matter that motivates this PR, but not on the merit of this PR. I would say this PR is the right thing to do. However, I think this is another example of how arrogant and insular the leadership of the Scala language team is. For further mind-bending context, see the [conversation ](https://contributors.scala-lang.org/t/proposal-to-deprecate-and-remove-symbol-literals/2953) on the proposal to remove the Symbol literal from Scala 2.13, which includes such gems of received wisdom as
   
   > Symbols are used in _some_ existing Scala code, but are not used pervasively.
   
   Apparently this PR does not count as pervasive usage. And apparently the symbol literal syntax in Spark SQL user code is not used pervasively. They know!
   
   IMO, Scala is (sadly) not a user-oriented programming language (anymore). It's an insular academic exercise. I pity the programmers who will be saddled with Scala 3.
   
   I do not mean to judge the value of this PR. Like I said, this rant is off topic. Scala 2.13 is what it and is. The ship has sailed. Under the circumstances of reality, this PR is the right thing to do. It's just sad it has to happen.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org