HyukjinKwon commented on a change in pull request #23664: [MINOR][DOCS] Add a 
note that 'spark.executor.pyspark.memory' is dependent on 'resource'
URL: https://github.com/apache/spark/pull/23664#discussion_r251273479
 
 

 ##########
 File path: docs/configuration.md
 ##########
 @@ -223,7 +225,8 @@ of the most common options to set are:
     stored on disk. This should be on a fast, local disk in your system. It 
can also be a
     comma-separated list of multiple directories on different disks.
 
-    NOTE: In Spark 1.0 and later this will be overridden by SPARK_LOCAL_DIRS 
(Standalone), MESOS_SANDBOX (Mesos) or
+    <br/>
+    <em>Note:</em> In Spark 1.0 and later this will be overridden by 
SPARK_LOCAL_DIRS (Standalone), MESOS_SANDBOX (Mesos) or
 
 Review comment:
   Fixing a nit around the code can be okay. This is the last instance in this 
page like:
   ![screen shot 2019-01-28 at 11 11 58 
am](https://user-images.githubusercontent.com/6477701/51812688-942bc200-22ed-11e9-9fcd-a02227bfa567.png)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to