Skip to content
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.

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

Add compression codec configurations for HoodieParquetWriter. #604

Merged

Conversation

kaka11chen
Copy link
Contributor

@kaka11chen kaka11chen commented Mar 15, 2019

Add compression codec configurations for HoodieParquetWriter
("hoodie.parquet.compression.codec" : "gzip" / "snappy" / "uncompressed" / "lzo")
https://issues.apache.org/jira/browse/HUDI-77

@vinothchandar vinothchandar self-requested a review March 15, 2019 18:04
Copy link
Member

@vinothchandar vinothchandar left a comment

Choose a reason for hiding this comment

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

Changes look great. Just one nit.

If you can also add modify docs (configurations.md) to add this new param in another PR, it will be awesome

@@ -44,6 +44,9 @@
public static final String PARQUET_COMPRESSION_RATIO = "hoodie.parquet.compression.ratio";
// Default compression ratio for parquet
public static final String DEFAULT_STREAM_COMPRESSION_RATIO = String.valueOf(0.1);
public static final String PARQUET_COMPRESSION_CODEC = "hoodie.parquet.compression.codec";
// Default compression ratio for parquet
public static final String DEFAULT_STREAM_COMPRESSION_CODEC = "gzip";
Copy link
Member

Choose a reason for hiding this comment

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

nit: consistently name to `DEFAULT_PARQUET_COMPRESSION_CODEC" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will change it, thx.

@kaka11chen
Copy link
Contributor Author

@vinothchandar Where is configurations.md?

@vinothchandar
Copy link
Member

https://hudi.apache.org/contributing.html#website talks about this.. its on the asf-site branch.. under docs/

@kaka11chen kaka11chen force-pushed the add-parquet-compression-codec-conf#77 branch from 896a9c5 to 81766e7 Compare March 15, 2019 18:14
@kaka11chen
Copy link
Contributor Author

@vinothchandar Thanks, will add document tomorrow.

@vinothchandar vinothchandar merged commit 48797b1 into apache:master Mar 18, 2019
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.

3 participants