Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2357#discussion_r192418603
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSession.scala ---
@@ -101,8 +100,8 @@ class CarbonSession(@transient val sc: SparkContext,
} catch {
case e: Exception =>
logError(String.format(
--- End diff --
It is in scala here, why still use `String.format`? Just use s"....$var..."---
