Skip to content

Conversation

BlueStalker
Copy link

as suggested in prestodb/presto#14464 , made change on these docker images to increase mem settings for hive

@BlueStalker
Copy link
Author

ping @arhimondr

@aweisberg aweisberg force-pushed the master branch 11 times, most recently from 44cb265 to 633e51b Compare February 26, 2021 18:41
Copy link
Member

@arhimondr arhimondr left a comment

Choose a reason for hiding this comment

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

LGTM % comments

@@ -0,0 +1 @@
export HADOOP_OPTS="$HADOOP_OPTS -Xmx1024m"
Copy link
Member

Choose a reason for hiding this comment

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

nit: new line at the end of the file (to keep Git happy)

@@ -0,0 +1 @@
export HADOOP_OPTS="$HADOOP_OPTS -Xmx1024m"
Copy link
Member

Choose a reason for hiding this comment

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

nit: new line at the end of the file (to keep Git happy)

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