Skip to content

[SPARK-25753][CORE]fix reading small files via BinaryFileRDD - #22725

Closed
10110346 wants to merge 1 commit into
apache:masterfrom
10110346:maxSplitSize_node_rack
Closed

[SPARK-25753][CORE]fix reading small files via BinaryFileRDD#22725
10110346 wants to merge 1 commit into
apache:masterfrom
10110346:maxSplitSize_node_rack

Conversation

@10110346

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This is a follow up of #21601, StreamFileInputFormat and WholeTextFileInputFormat have the same problem.

Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304 java.io.IOException: Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304 at org.apache.hadoop.mapreduce.lib.input.CombineFileInputFormat.getSplits(CombineFileInputFormat.java: 201) at org.apache.spark.rdd.BinaryFileRDD.getPartitions(BinaryFileRDD.scala:52) at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:254) at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:252) at scala.Option.getOrElse(Option.scala:121) at org.apache.spark.rdd.RDD.partitions(RDD.scala:252) at org.apache.spark.SparkContext.runJob(SparkContext.scala:2138)

How was this patch tested?

Added a unit test

@SparkQA

Copy link
Copy Markdown

Test build #97378 has finished for PR 22725 at commit 54ffcdb.

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

@10110346

Copy link
Copy Markdown
ContributorAuthor

cc @dhruve@tgravescs

@tgravescs

Copy link
Copy Markdown
Contributor

SPARK-24610 is the original issue, please file a new jira for StreamFileInputFormat

@10110346

Copy link
Copy Markdown
ContributorAuthor

@tgravescs ok, I will do it ,thanks

@1011034610110346 changed the title [SPARK-24610][[CORE][FOLLOW-UP]fix reading small files via BinaryFileRDD[SPARK-25753][[CORE][FOLLOW-UP]fix reading small files via BinaryFileRDDOct 17, 2018
@tgravescs

Copy link
Copy Markdown
Contributor

+1 Looks good, thanks @10110346

@dongjoon-hyundongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @10110346 . Could you change the title?

- [SPARK-25753][[CORE][FOLLOW-UP]fix reading small files via BinaryFileRDD
+ [SPARK-25753][CORE] Fix reading small files via BinaryFileRDD

@10110346

Copy link
Copy Markdown
ContributorAuthor

ok,thanks @dongjoon-hyun

@1011034610110346 changed the title [SPARK-25753][[CORE][FOLLOW-UP]fix reading small files via BinaryFileRDD[SPARK-25753][[CORE]fix reading small files via BinaryFileRDDOct 20, 2018
@dongjoon-hyun

Copy link
Copy Markdown
Member

It still has [[ before CORE. :)

@1011034610110346 changed the title [SPARK-25753][[CORE]fix reading small files via BinaryFileRDD[SPARK-25753][CORE]fix reading small files via BinaryFileRDDOct 20, 2018
@tgravescs

Copy link
Copy Markdown
Contributor

merged to master

@SparkQA

Copy link
Copy Markdown

Test build #97854 has started for PR 22725 at commit 54ffcdb.

@SparkQA

Copy link
Copy Markdown

Test build #97862 has started for PR 22725 at commit 54ffcdb.

@SparkQA

Copy link
Copy Markdown

Test build #97874 has started for PR 22725 at commit 54ffcdb.

@AmplabJenkins

Copy link
Copy Markdown

Build finished. Test FAILed.

jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?
This is a follow up of apache#21601, `StreamFileInputFormat` and `WholeTextFileInputFormat` have the same problem.
`Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304
java.io.IOException: Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304
at org.apache.hadoop.mapreduce.lib.input.CombineFileInputFormat.getSplits(CombineFileInputFormat.java: 201)
at org.apache.spark.rdd.BinaryFileRDD.getPartitions(BinaryFileRDD.scala:52)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:254)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:252)
at scala.Option.getOrElse(Option.scala:121)
at org.apache.spark.rdd.RDD.partitions(RDD.scala:252)
at org.apache.spark.SparkContext.runJob(SparkContext.scala:2138)`
## How was this patch tested?
Added a unit test
Closesapache#22725 from 10110346/maxSplitSize_node_rack.
Authored-by: liuxian <liu.xian3@zte.com.cn>
Signed-off-by: Thomas Graves <tgraves@apache.org>
dhruve pushed a commit to dhruve/spark that referenced this pull request Oct 4, 2019
## What changes were proposed in this pull request?
This is a follow up of apache#21601, `StreamFileInputFormat` and `WholeTextFileInputFormat` have the same problem.
`Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304
java.io.IOException: Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304
at org.apache.hadoop.mapreduce.lib.input.CombineFileInputFormat.getSplits(CombineFileInputFormat.java: 201)
at org.apache.spark.rdd.BinaryFileRDD.getPartitions(BinaryFileRDD.scala:52)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:254)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:252)
at scala.Option.getOrElse(Option.scala:121)
at org.apache.spark.rdd.RDD.partitions(RDD.scala:252)
at org.apache.spark.SparkContext.runJob(SparkContext.scala:2138)`
## How was this patch tested?
Added a unit test
Closesapache#22725 from 10110346/maxSplitSize_node_rack.
Authored-by: liuxian <liu.xian3@zte.com.cn>
Signed-off-by: Thomas Graves <tgraves@apache.org>
dongjoon-hyun pushed a commit that referenced this pull request Oct 4, 2019
### What changes were proposed in this pull request?
This is a clean cherry pick of #22725 from master to 2.4
This is a follow up of #21601, `StreamFileInputFormat` and `WholeTextFileInputFormat` have the same problem.
`Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304
java.io.IOException: Minimum split size pernode 5123456 cannot be larger than maximum split size 4194304
at org.apache.hadoop.mapreduce.lib.input.CombineFileInputFormat.getSplits(CombineFileInputFormat.java: 201)
at org.apache.spark.rdd.BinaryFileRDD.getPartitions(BinaryFileRDD.scala:52)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:254)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:252)
at scala.Option.getOrElse(Option.scala:121)
at org.apache.spark.rdd.RDD.partitions(RDD.scala:252)
at org.apache.spark.SparkContext.runJob(SparkContext.scala:2138)`
### Why are the changes needed?
This is an existing bug which was fixed in master, but not back ported to 2.4.
### Does this PR introduce any user-facing change?
No
### How was this patch tested?
The original patch added a unit test.
Ran the unit test that was added in the original patch and manually verified the changes by creating a multiline csv and loading it in spark shell.
Closes#26026 from dhruve/fix/SPARK-25753/2.4.
Authored-by: liuxian <liu.xian3@zte.com.cn>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
Sign up for freeto 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.

5 participants

@10110346@SparkQA@tgravescs@dongjoon-hyun@AmplabJenkins