srowen commented on issue #22570: [SPARK-25553][BUILD] Add EmptyInterpolatedStringChecker to scalastyle config URL: https://github.com/apache/spark/pull/22570#issuecomment-531589388 I'm OK dropping this on the grounds that it makes backports harder, even though this is for Spark 3. The argument for the change is a tiny bit of performance, but it's probably negligible except perhaps in a few hot paths. I wonder if there is a small subset of these changes that look like they could matter at all for perf and just fix those?
---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
