leovegas commented on code in PR #46678:
URL: https://github.com/apache/spark/pull/46678#discussion_r1607884919
##########
core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala:
##########
@@ -87,8 +88,8 @@ private[spark] class SparkHadoopUtil extends Logging {
}
/**
- * Appends spark.hadoop.* configurations from a [[SparkConf]] to a Hadoop
- * configuration without the spark.hadoop. prefix.
+ * Appends spark.hadoop.* configurations from a [[SparkConf]] to a Hadoop
configuration without
Review Comment:
Should I apply fmt only to the new code? That unclear
##########
core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala:
##########
@@ -142,15 +143,16 @@ private[spark] class SparkHadoopUtil extends Logging {
if (!new File(keytabFilename).exists()) {
throw new SparkException(s"Keytab file: ${keytabFilename} does not
exist")
} else {
- logInfo("Attempting to login to Kerberos " +
- s"using principal: ${principalName} and keytab: ${keytabFilename}")
+ logInfo(
Review Comment:
Rolled back unnecessary formatting
--
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]
