[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-20 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23085 @mrandrewandrade, let's close this for now. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-20 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23085 Because it's already documented. Also it brings maintnense overhead. --- - To unsubscribe, e-mail:

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-20 Thread mrandrewandrade
Github user mrandrewandrade commented on the issue: https://github.com/apache/spark/pull/23085 Why wouldn't you update the documentation to allow users to understand what file formats they can use? It's really bad practice for users to read source code. ---

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23085 It's already documented in official site anyway. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-19 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/23085 I think it doesn't need to change. It's likely to be changed and we wouldn't want to update this doc everytime we add new datasource. ---

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23085 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-19 Thread mrandrewandrade
Github user mrandrewandrade commented on the issue: https://github.com/apache/spark/pull/23085 Since it's a trivial doc change, I didn't make a Jira ticket. Please let me know if there are any changes required. ---

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23085 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #23085: [Docs] Added csv, orc, and text output format options to...

2018-11-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/23085 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional