Skip to content

Commit 625d39f

Browse files
author
Siri Teja Gaddameedi
committed
Sanitize sparkUpgrade error message
Signed-off-by: Siri Teja Gaddameedi <[email protected]>
1 parent b9a0dc5 commit 625d39f

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

spark-sql-application/src/main/scala/org/apache/spark/sql/JobOperator.scala

+11
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ import org.opensearch.flint.common.scheduler.model.LangType
1717
import org.opensearch.flint.core.metrics.{MetricConstants, MetricsSparkListener, MetricsUtil}
1818
import org.opensearch.flint.spark.FlintSpark
1919

20+
import org.apache.spark.SparkUpgradeException
2021
import org.apache.spark.internal.Logging
2122
import org.apache.spark.sql.flint.config.FlintSparkConf
2223
import org.apache.spark.sql.util.ShuffleCleaner
@@ -153,6 +154,11 @@ case class JobOperator(
153154
}
154155
})
155156
} catch {
157+
case e: SparkUpgradeException =>
158+
incrementCounter(MetricConstants.RESULT_WRITER_FAILED_METRIC)
159+
throwableHandler.recordThrowable(
160+
s"Failed to write to result. Cause='${sanitizeSparkUpgradeErrorMessage(e.getMessage)}'",
161+
e)
156162
case t: Throwable =>
157163
incrementCounter(MetricConstants.RESULT_WRITER_FAILED_METRIC)
158164
throwableHandler.recordThrowable(
@@ -320,6 +326,11 @@ case class JobOperator(
320326
metricName
321327
}
322328

329+
private def sanitizeSparkUpgradeErrorMessage(errorMessage: String): String = {
330+
val pattern = "You may get a different result due to the upgrading to Spark[^\\n]*:\\n".r
331+
pattern.replaceAllIn(errorMessage, "").trim
332+
}
333+
323334
private def incrementCounter(metricName: String): Unit = {
324335
MetricsUtil.incrementCounter(resolveMetricName(metricName))
325336
}

0 commit comments

Comments
 (0)