[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19596 Thank you @rxin and @viirya. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user rxin commented on the issue: https://github.com/apache/spark/pull/19596 Merging in master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19596 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19596 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83334/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19596 **[Test build #83334 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83334/testReport)** for PR 19596 at commit [`82fe78b`](https://github.com/apache/spark/commit/82fe78bd6f16fabcc82e0020e0d1b77bc3b5a18f). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19596 **[Test build #83334 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83334/testReport)** for PR 19596 at commit [`82fe78b`](https://github.com/apache/spark/commit/82fe78bd6f16fabcc82e0020e0d1b77bc3b5a18f). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19596 Thanks, @viirya. I wasn't sure if I should add it to the list. My intention was .. this one is like `DataFrameReader` and `DataFrameWriter` (supposed to be used via `spark.read`) and I wanted to.. like hide the package path `pyspark.sql.Catalog` in the doc and, I just decided the smallest change I could think for this issue. I am fine with adding it too. It's easy to add it if anyone feels strongly about this. Please let me know. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user viirya commented on the issue: https://github.com/apache/spark/pull/19596 I've generated the Python docs. Looks good. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user viirya commented on the issue: https://github.com/apache/spark/pull/19596 Don't we like to add it to the list of `Important classes of Spark SQL and DataFrames`? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19596 cc @holdenk and @viirya, mind taking a look please? I remember I had few talks about Sphinx and `__all__` and I believe you guys are right reviewers. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user rxin commented on the issue: https://github.com/apache/spark/pull/19596 Yea definitely. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19596 cc @rxin, do you think it's okay to expose PySpark's Catalog class API doc? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19596 **[Test build #83170 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83170/testReport)** for PR 19596 at commit [`82fe78b`](https://github.com/apache/spark/commit/82fe78bd6f16fabcc82e0020e0d1b77bc3b5a18f). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19596 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83170/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19596 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19596: [SPARK-22369][PYTHON][DOCS] Exposes catalog API document...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19596 **[Test build #83170 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83170/testReport)** for PR 19596 at commit [`82fe78b`](https://github.com/apache/spark/commit/82fe78bd6f16fabcc82e0020e0d1b77bc3b5a18f). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org