Skip to content

Conversation

@shivusondur
Copy link
Contributor

@shivusondur shivusondur commented Oct 8, 2018

What changes were proposed in this pull request?

Documentation is updated with proper classname org.apache.spark.io.ZStdCompressionCodec

How was this patch tested?

we used the spark.io.compression.codec = org.apache.spark.io.ZStdCompressionCodec
and verified the logs.

…ache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException Exception

Documentation is updated with proper classname org.apache.spark.io.ZstdCompressionCodec
@shivusondur shivusondur changed the title [SPARK-25677] [Spark Compression] spark.io.compression.codec = org.apache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException Exception [SPARK-25677] [Core] spark.io.compression.codec = org.apache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException Exception Oct 8, 2018
@wangyum
Copy link
Member

wangyum commented Oct 8, 2018

Please update title to:

[SPARK-25677][DOC] ...

@wangyum
Copy link
Member

wangyum commented Oct 8, 2018

cc @HyukjinKwon

@shivusondur shivusondur changed the title [SPARK-25677] [Core] spark.io.compression.codec = org.apache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException Exception [SPARK-25677] [Doc] spark.io.compression.codec = org.apache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException Exception Oct 8, 2018
Copy link
Member

@felixcheung felixcheung left a comment

Choose a reason for hiding this comment

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

can you double check if there is any other copy?

@shivusondur
Copy link
Contributor Author

@felixcheung I searched in all the code base, only this place i found

@HyukjinKwon
Copy link
Member

ok to test

@SparkQA
Copy link

SparkQA commented Oct 8, 2018

Test build #97103 has finished for PR 22669 at commit 8a1a30d.

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

@HyukjinKwon
Copy link
Member

Merged to master and branch-2.4.

asfgit pushed a commit that referenced this pull request Oct 8, 2018
…stdCompressionCodec throwing IllegalArgumentException Exception

## What changes were proposed in this pull request?
Documentation is updated with proper classname org.apache.spark.io.ZStdCompressionCodec

## How was this patch tested?
we used the  spark.io.compression.codec = org.apache.spark.io.ZStdCompressionCodec
and verified the logs.

Closes #22669 from shivusondur/CompressionIssue.

Authored-by: shivusondur <[email protected]>
Signed-off-by: hyukjinkwon <[email protected]>
(cherry picked from commit 1a6815c)
Signed-off-by: hyukjinkwon <[email protected]>
@asfgit asfgit closed this in 1a6815c Oct 8, 2018
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
…stdCompressionCodec throwing IllegalArgumentException Exception

## What changes were proposed in this pull request?
Documentation is updated with proper classname org.apache.spark.io.ZStdCompressionCodec

## How was this patch tested?
we used the  spark.io.compression.codec = org.apache.spark.io.ZStdCompressionCodec
and verified the logs.

Closes apache#22669 from shivusondur/CompressionIssue.

Authored-by: shivusondur <[email protected]>
Signed-off-by: hyukjinkwon <[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.

5 participants