HyukjinKwon commented on a change in pull request #33740:
URL: https://github.com/apache/spark/pull/33740#discussion_r693418067



##########
File path: docs/configuration.md
##########
@@ -659,6 +659,16 @@ Apart from these, the following properties are also 
available, and may be useful
   </td>
   <td>2.1.2</td>
 </tr>
+<tr>
+  <td><code>spark.redaction.string.regex</code></td>
+  <td>(none)</td>
+  <td>
+    Regex to decide which parts of strings produced by Spark contain sensitive
+    information. When this regex matches a string part, that string part is 
replaced by a
+    dummy value. This is currently used to redact the output of SQL explain 
commands.

Review comment:
       I am not sure on this for now. If we happen to add the redaction on 
other places not in core, `spark.redaction.string.regex` could make sense. We 
won't necessarily have to deprecate it - I would just prefer to leave it 
undocumented because semantically it looks fine now although the configuration 
is virtually useless at this moment.




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