skonto commented on a change in pull request #23993: [SPARK-26742][K8s]Update
k8s client version to 4.1.2 for Spark 2.4.1
URL: https://github.com/apache/spark/pull/23993#discussion_r263085388
##########
File path:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/LoggingPodStatusWatcher.scala
##########
@@ -174,7 +174,7 @@ private[k8s] class LoggingPodStatusWatcherImpl(
}.getOrElse(Seq(("Container state", "N/A")))
}
- private def formatTime(time: Time): String = {
- if (time != null) time.getTime else "N/A"
+ private def formatTime(time: String): String = {
+ if (time != null || time != "") time else "N/A"
Review comment:
this part of the codebase has changed on master, but I am not porting back
all the changes, just fixing this locally.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
