Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/19632#discussion_r148370955 --- Diff: examples/src/main/python/pi.py --- @@ -27,12 +27,16 @@ if __name__ == "__main__": """ Usage: pi [partitions] + + Monte Carlo method is used to estimate Pi in the below example. """ spark = SparkSession\ .builder\ .appName("PythonPi")\ .getOrCreate() - + + # If no arguments are passed(i.e. `len(sys.argv) < = 1` ) --- End diff -- This isn't visible to end users, so don't know if this matters. It's already documented in the usage. I am not sure this meaningfully helps.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org