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



##########
File path: docs/running-on-kubernetes.md
##########
@@ -1083,14 +1083,6 @@ See the [configuration page](configuration.html) for 
information on Spark config
   </td>
   <td>2.4.0</td>
 </tr>
-<tr>
-  <td><code>spark.kubernetes.pyspark.pythonVersion</code></td>
-  <td><code>"3"</code></td>
-  <td>
-   This sets the major Python version of the docker image used to run the 
driver and executor containers. Can be 3.
-  </td>
-  <td>2.4.0</td>
-</tr>

Review comment:
       @erikerlandson, just to extra clarify, there's no need to `@deprecated`. 
The configuration instance is not an API. It's `private[spark]`. It would have 
been nicer to clarify what's the actual gain of deprecating over removing a 
no-op configuration. 




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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to