GitHub user lw-lin opened a pull request:

    https://github.com/apache/spark/pull/13518

    [SPARK-15472][SQL] Add support for writing in `csv`, `json`, `text` formats 
in Structured Streaming

    ## What changes were proposed in this pull request?
    
    This patch adds support for writing in `csv`, `json`, `text` formats in 
Structured Streaming:
    
    **1. at a high level, this patch forms the following hierarchy**(`text` as 
an example):
    ```
                    <OutputWriter>
                          ↑
                 TextOutputWriterBase
                     ↗          ↖
    BatchTextOutputWriter   StreamingTextOutputWriter
    ```
    ```
                        <OutputWriterFactory>
                            ↗          ↖
    BatchTextOutputWriterFactory   StreamingOutputWriterFactory
                                                  ↑
                                  StreamingTextOutputWriterFactory
    ```
    The `StreamingTextOutputWriter` and other 'streaming' output writers would 
write data **without** using an `OutputCommitter`. This was the same approach 
taken by [SPARK-14716](https://github.com/apache/spark/pull/12409).
    
    **2. to support compression, this patch attaches an extension to the path 
assigned by `FileStreamSink`.** For example, if we are writing out using the 
`gzip` compression and `FileStreamSink` assigns path `{$uuid}` to a text 
writer, then in the end the file written out will be `${uuid}.txt.gz` -- so 
that when we read the file back, we'll correctly interpret it as `gzip` 
compressed.
    
    
     ## How was this patch tested?
    
    `FileStreamSinkSuite` is much more expanded to cover the added `csv`, 
`json`, `text` format:
    
    ```scala
    test(" csv - unpartitioned data - codecs: none/gzip")
    test("json - unpartitioned data - codecs: none/gzip")
    test("text - unpartitioned data - codecs: none/gzip")
    
    test(" csv - partitioned data - codecs: none/gzip")
    test("json - partitioned data - codecs: none/gzip")
    test("text - partitioned data - codecs: none/gzip")
    
    test(" csv - unpartitioned writing and batch reading - codecs: none/gzip")
    test("json - unpartitioned writing and batch reading - codecs: none/gzip")
    test("text - unpartitioned writing and batch reading - codecs: none/gzip")
    
    test(" csv - partitioned writing and batch reading - codecs: none/gzip")
    test("json - partitioned writing and batch reading - codecs: none/gzip")
    test("text - partitioned writing and batch reading - codecs: none/gzip")
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/lw-lin/spark add-csv-json-text-for-ss

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/13518.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #13518
    
----
commit 97034f9aeb092b10e1606e60a8e6b4878ebd54cf
Author: Liwei Lin <lwl...@gmail.com>
Date:   2016-06-05T09:03:04Z

    Add csv, json, text

commit 2035b597b44aa519d8da3b155036446f88b3050e
Author: Liwei Lin <lwl...@gmail.com>
Date:   2016-06-05T09:03:15Z

    Fix parquet extension

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to