Skip to content

[SPARK-29542][SQL][DOC] Make the descriptions of spark.sql.files.* be clearly #26200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
turboFei wants to merge 6 commits into from
Closed

[SPARK-29542][SQL][DOC] Make the descriptions of spark.sql.files.* be clearly #26200

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
dd28b09
[SPARK-29542] Make the description of spark.sql.files.maxPartitionByt…
turboFei Oct 22, 2019
7ecb7b6
fix style
turboFei Oct 22, 2019
4a56b2e
fix blank
turboFei Oct 22, 2019
849e995
revert
turboFei Oct 23, 2019
1de48df
refactor
turboFei Oct 23, 2019
7e115d8
revert maxRecordsPerFile
turboFei Oct 23, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 11 additions & 4 deletions sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -980,7 +980,9 @@ object SQLConf {
.createWithDefault(true)

val FILES_MAX_PARTITION_BYTES = buildConf("spark.sql.files.maxPartitionBytes")
.doc("The maximum number of bytes to pack into a single partition when reading files.")
.doc("The maximum number of bytes to pack into a single partition when reading files. " +
"This configuration is effective only when using file-based sources such as Parquet, JSON " +
"and ORC.")
.bytesConf(ByteUnit.BYTE)
.createWithDefault(128 * 1024 * 1024) // parquet.block.size

Expand All @@ -989,19 +991,24 @@ object SQLConf {
.doc("The estimated cost to open a file, measured by the number of bytes could be scanned in" +
" the same time. This is used when putting multiple files into a partition. It's better to" +
" over estimated, then the partitions with small files will be faster than partitions with" +
" bigger files (which is scheduled first).")
" bigger files (which is scheduled first). This configuration is effective only when using" +
" file-based sources such as Parquet, JSON and ORC.")
.longConf
.createWithDefault(4 * 1024 * 1024)

val IGNORE_CORRUPT_FILES = buildConf("spark.sql.files.ignoreCorruptFiles")
.doc("Whether to ignore corrupt files. If true, the Spark jobs will continue to run when " +
"encountering corrupted files and the contents that have been read will still be returned.")
"encountering corrupted files and the contents that have been read will still be returned. " +
"This configuration is effective only when using file-based sources such as Parquet, JSON " +
"and ORC.")
.booleanConf
.createWithDefault(false)

val IGNORE_MISSING_FILES = buildConf("spark.sql.files.ignoreMissingFiles")
.doc("Whether to ignore missing files. If true, the Spark jobs will continue to run when " +
"encountering missing files and the contents that have been read will still be returned.")
"encountering missing files and the contents that have been read will still be returned. " +
"This configuration is effective only when using file-based sources such as Parquet, JSON " +
"and ORC.")
.booleanConf
.createWithDefault(false)

Expand Down