Documentation Updates for New Write Related Features#7520
Merged
alamb merged 11 commits intoapache:mainfrom Sep 12, 2023
Merged
Documentation Updates for New Write Related Features#7520alamb merged 11 commits intoapache:mainfrom
alamb merged 11 commits intoapache:mainfrom
Conversation
Weijun-H
reviewed
Sep 11, 2023
Member
Weijun-H
left a comment
Weijun-H
left a comment
There was a problem hiding this comment.
Hello @devinjdangelo, I noticed some typos in this pr.
|
|
||
| | Option | Description | Default Value | | ||
| | ----------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ | ---------------------------------------------------------------------------- | | ||
| | SINGLE_FILE | If true, indicates that this external table is backed by a single file. INSERT INTO queries will append to this file. | false | |
Member
There was a problem hiding this comment.
Suggested change
| | SINGLE_FILE | If true, indicates that this external table is backed by a single file. INSERT INTO queries will append to this file. | false | | |
| | SINGLE_FILE | If true, indicates that this external table is backed by a single file. INSERT INTO queries will be appended to this file. | false | |
| ) | ||
| ``` | ||
|
|
||
| In this example, we write the entirety of `source_table` out to a folder of parquet files. The option `single_file_output` set to false, indicates that the destination path should be interpreted as a folder to which the query will output multiple files. One parquet file will be written in parallel to the folder for each partition in the query. The next option `compression` set to `snappy` indicates that unless otherwise specified all columns should use the snappy compression codec. The option `compression::col1` sets an override, so that the column `col1` in the parquet file will use `ZSTD` compression codec with compression level `5`. In general, parquet option which support column specific settings can be specified with the syntax `OPTION::COLUMN.NESTED.PATH`. |
Member
There was a problem hiding this comment.
Suggested change
| In this example, we write the entirety of `source_table` out to a folder of parquet files. The option `single_file_output` set to false, indicates that the destination path should be interpreted as a folder to which the query will output multiple files. One parquet file will be written in parallel to the folder for each partition in the query. The next option `compression` set to `snappy` indicates that unless otherwise specified all columns should use the snappy compression codec. The option `compression::col1` sets an override, so that the column `col1` in the parquet file will use `ZSTD` compression codec with compression level `5`. In general, parquet option which support column specific settings can be specified with the syntax `OPTION::COLUMN.NESTED.PATH`. | |
| In this example, we write the entirety of `source_table` out to a folder of parquet files. The option `single_file_output` set to false, indicates that the destination path should be interpreted as a folder to which the query will output multiple files. One parquet file will be written in parallel to the folder for each partition in the query. The next option `compression` set to `snappy` indicates that unless otherwise specified all columns should use the snappy compression codec. The option `compression::col1` sets an override, so that the column `col1` in the parquet file will use `ZSTD` compression codec with compression level `5`. In general, the parquet option which supports column-specific settings can be specified with the syntax `OPTION::COLUMN.NESTED.PATH`. |
Weijun-H
reviewed
Sep 11, 2023
Member
Weijun-H
left a comment
Weijun-H
left a comment
There was a problem hiding this comment.
Hello @devinjdangelo, I noticed some typos in this pr.
alamb
approved these changes
Sep 11, 2023
Contributor
alamb
left a comment
alamb
left a comment
There was a problem hiding this comment.
Thank you @devinjdangelo -- this looks really great ❤️
Thank you @Weijun-H for the additional review
| files in the `dir_name` directory: | ||
|
|
||
| ```sql | ||
| > COPY source_table TO 'dir_name' (FORMAT parquet, PER_THREAD_OUTPUT true); |
| under the License. | ||
| --> | ||
|
|
||
| # Write Options |
Contributor
There was a problem hiding this comment.
I think it might be a good idea to add a link to this page into the index https://github.com/apache/arrow-datafusion/blob/main/docs/source/user-guide/sql/index.rst so it show up in the left hand nav bar
Co-authored-by: Andrew Lamb <[email protected]>
Contributor
|
Thanks again @devinjdangelo and @Weijun-H -- I'll merge this and we can continue iterating on the docs in follow on PRs |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
Closes #7499
Rationale for this change
We have added new options for writing files and changed some names around. We should update the documentation so the current state is clear.
What changes are included in this PR?
New documentation for write related options.
Are these changes tested?
Yes by existing tests.
Are there any user-facing changes?
New docs