zhengruifeng commented on code in PR #48221: URL: https://github.com/apache/spark/pull/48221#discussion_r1772805433
########## sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala: ########## @@ -3171,9 +3171,9 @@ object SQLConf { val PYSPARK_PLOT_MAX_ROWS = buildConf("spark.sql.pyspark.plotting.max_rows") - .doc( - "The visual limit on top-n-based plots. If set to 1000, the first 1000 data points " + - "will be used for plotting.") + .doc("The visual limit on plots. If set to 1000 for top-n-based plots, the first 1000 " + Review Comment: shall we also list the plots of `top-n-based plots` and `sampled-based plots` here? e.g. `If set to 1000 for top-n-based plots (pie, bar, barh), ...` -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org