holdenk commented on code in PR #58620:
URL: https://github.com/apache/spark/pull/58620#discussion_r4078165987


##########
core/src/main/scala/org/apache/spark/internal/config/Deploy.scala:
##########
@@ -51,6 +51,19 @@ private[spark] object Deploy {
     .checkValue(_ > 0, "spark.deploy.recoveryTimeout must be positive.")
     .createOptional
 
+  val RECOVERY_SERIALIZATION_FILTER =
+    ConfigBuilder("spark.deploy.recoverySerializationFilter")
+      .doc("JEP-290 serialization filter pattern applied when the master reads 
back " +
+        "recovery state written by the built-in JavaSerializer (currently 
enforced for " +
+        "the ZOOKEEPER recovery mode). The default allows only JDK, Scala and 
Spark " +
+        "classes, which covers everything the master persists 
(ApplicationInfo, " +
+        "DriverInfo, WorkerInfo and their fields). Znodes containing any other 
class " +
+        "are skipped during recovery instead of being instantiated in the 
newly " +
+        "elected master. Set to '*' to disable filtering.")
+      .version("4.3.0")
+      .stringConf
+      .createWithDefault("java.**;scala.**;org.apache.spark.**;!*")

Review Comment:
   Did a slightly more involved checkValue :)



-- 
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]

Reply via email to