drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sudheeshkatkam <...@git.apache.org>
Subject [GitHub] drill pull request #547: Drill-4581: Extensive revisions to the Drill launch...
Date Thu, 21 Jul 2016 01:02:49 GMT
Github user sudheeshkatkam commented on a diff in the pull request:

    --- Diff: distribution/src/resources/drill-config.sh ---
    @@ -193,21 +366,24 @@ if $is_cygwin; then
       DRILL_LOG_DIR=`cygpath -w "$DRILL_LOG_DIR"`
       CP=`cygpath -w -p "$CP"`
       if [ -z "$HADOOP_HOME" ]; then
    -    HADOOP_HOME=${DRILL_HOME}/winutils
    +    export HADOOP_HOME=${DRILL_HOME}/winutils
     # make sure allocator chunks are done as mmap'd memory (and reduce arena overhead)
    -export MALLOC_ARENA_MAX=4
    +# Newer versions of glibc use an arena memory allocator that causes virtual
    +# memory usage to explode. Tune the variable down to prevent vmem explosion.
     export MALLOC_MMAP_THRESHOLD_=131072
     export MALLOC_TRIM_THRESHOLD_=131072
     export MALLOC_TOP_PAD_=131072
     export MALLOC_MMAP_MAX_=65536
     # Variables exported form this script
    -export HADOOP_HOME
     export is_cygwin
     export DRILL_HOME
     export DRILL_CONF_DIR
     export DRILL_LOG_DIR
     export CP
    +export JAVA_HOME
    +export JAVA
    --- End diff --
    Why these?

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.

View raw message