Skip to content

Conversation

@Ngone51
Copy link
Member

@Ngone51 Ngone51 commented Nov 27, 2019

What changes were proposed in this pull request?

Improve conf IO_WARNING_LARGEFILETHRESHOLD (a.k.a spark.io.warning.largeFileThreshold):

  • reword documentation

  • change type from long to bytes

Why are the changes needed?

Improvements according to #25134 (comment) & #25134 (comment).

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass Jenkins.

@Ngone51
Copy link
Member Author

Ngone51 commented Nov 27, 2019

cc @gatorsmile @cloud-fan

@SparkQA
Copy link

SparkQA commented Nov 27, 2019

Test build #114516 has finished for PR 26691 at commit a24b273.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in d075b33 Nov 27, 2019
attilapiros pushed a commit to attilapiros/spark that referenced this pull request Dec 6, 2019
…HRESHOLD

### What changes were proposed in this pull request?

Improve conf `IO_WARNING_LARGEFILETHRESHOLD` (a.k.a `spark.io.warning.largeFileThreshold`):

* reword documentation

* change type from `long` to `bytes`

### Why are the changes needed?

Improvements according to apache#25134 (comment) & apache#25134 (comment).

### Does this PR introduce any user-facing change?

No.

### How was this patch tested?

Pass Jenkins.

Closes apache#26691 from Ngone51/SPARK-28366-followup.

Authored-by: wuyi <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants