mridulm commented on code in PR #48440: URL: https://github.com/apache/spark/pull/48440#discussion_r1887581440
########## common/utils/src/main/scala/org/apache/spark/SparkException.scala: ########## @@ -24,20 +24,14 @@ import java.util.ConcurrentModificationException import scala.jdk.CollectionConverters._ -class SparkException( +class SparkException private( Review Comment: I will let the other reviewers comment on this more, since I am not following the evolution of error classes closely. But I would encourage ensuring this is in line with [versioning policy](https://spark.apache.org/versioning-policy.html), and we minimize breaking api's. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
