move safe logging to internal

Author: yifeih

core/pom.xml

@@ -365,6 +365,7 @@
       <artifactId>spark-tags_${scala.binary.version}</artifactId>
     </dependency>
 
+
     <dependency>
       <groupId>org.apache.spark</groupId>
       <artifactId>spark-launcher_${scala.binary.version}</artifactId>
@@ -416,6 +417,15 @@
       <scope>provided</scope>
     </dependency>
 
+    <!--
+      The following dependency is used for safe logging
+    -->
+    <dependency>
+      <groupId>com.palantir.safe-logging</groupId>
+      <artifactId>safe-logging</artifactId>
+      <version>1.5.1</version>
+    </dependency>
+
   </dependencies>
   <build>
     <outputDirectory>target/scala-${scala.binary.version}/classes</outputDirectory>

resource-managers/kubernetes/core/src/main/scala/org/apache/spark/logging/SafeLogging.scala renamed to core/src/main/scala/org/apache/spark/internal/SafeLogging.scala

@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-package org.apache.spark.logging
+package org.apache.spark.internal
 
 import com.palantir.logsafe.Arg
 import org.slf4j.LoggerFactory
@@ -63,3 +63,4 @@ trait SafeLogging {
     if (log_.isErrorEnabled) log_.error(message, args :+ error: _*)
   }
 }
+

resource-managers/kubernetes/core/pom.xml

@@ -89,12 +89,6 @@
       <version>3.8.1</version>
     </dependency>
 
-    <dependency>
-      <groupId>com.palantir.safe-logging</groupId>
-      <artifactId>safe-logging</artifactId>
-      <version>1.5.1</version>
-    </dependency>
-
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>

resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala

@@ -27,7 +27,7 @@ import org.apache.spark.{SparkConf, SparkException}
 import org.apache.spark.deploy.k8s.Config._
 import org.apache.spark.deploy.k8s.Constants._
 import org.apache.spark.deploy.k8s.KubernetesConf
-import org.apache.spark.logging.SafeLogging
+import org.apache.spark.internal.SafeLogging
 import org.apache.spark.util.{Clock, Utils}
 
 private[spark] class ExecutorPodsAllocator(

resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsLifecycleManager.scala

@@ -26,7 +26,7 @@ import scala.collection.mutable
 import org.apache.spark.SparkConf
 import org.apache.spark.deploy.k8s.Config._
 import org.apache.spark.deploy.k8s.KubernetesUtils._
-import org.apache.spark.logging.SafeLogging
+import org.apache.spark.internal.SafeLogging
 import org.apache.spark.scheduler.ExecutorExited
 import org.apache.spark.util.Utils