Skip to content

Enable block gzip compression example#85

Open
sagehen03 wants to merge 2 commits intomasterfrom
dh-block-gzip-example
Open

Enable block gzip compression example#85
sagehen03 wants to merge 2 commits intomasterfrom
dh-block-gzip-example

Conversation

@sagehen03
Copy link
Contributor

@sagehen03 sagehen03 commented Apr 18, 2023

Making this work comes down to specifying the codec when writing from pyspark and also having that codec on the Spark classpath.

@@ -0,0 +1,3 @@
#!/bin/bash -xe

sudo aws s3 cp s3://dig-data-registry/hail.jar /usr/lib/spark/jars/
Copy link
Contributor Author

@sagehen03 sagehen03 Apr 18, 2023

Choose a reason for hiding this comment

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

I built the hail.jar from the hail source (it needs to compiled using java 8 since that's what our EMR clusters use) and then uploaded it to S3. We probably need a better location in s3, but I used this one now since it's not in production use. This solution also relies on EMR continuing to put /usr/lib/spark/jars on the classpath.

Copy link
Contributor

Choose a reason for hiding this comment

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

So for things like this we tend to use s3://dig-aggregator-data/bin/

Copy link
Contributor

@psmadbec psmadbec left a comment

Choose a reason for hiding this comment

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

Magical. For the generation of the hail.jar either we'll want instructions, or probably we'll want something to be put into dig-analysis-data/scripts which is where I put things that have a specific generation sequence to help in case someone needs to generate or update the file itself.

@@ -0,0 +1,3 @@
#!/bin/bash -xe

sudo aws s3 cp s3://dig-data-registry/hail.jar /usr/lib/spark/jars/
Copy link
Contributor

Choose a reason for hiding this comment

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

So for things like this we tend to use s3://dig-aggregator-data/bin/

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.

2 participants