Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/647#discussion_r86916748
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ---
    @@ -49,6 +49,7 @@
       String USER_RPC_TIMEOUT = "drill.exec.rpc.user.timeout";
       String METRICS_CONTEXT_NAME = "drill.exec.metrics.context";
       String USE_IP_ADDRESS = "drill.exec.rpc.use.ip";
    +  String BIT_ADVERTISED_HOST = "drill.exec.rpc.bit.advertised.host";
    --- End diff --
    
    drill-env.sh is a script that holds the "external" customizations for each 
installation. "External" customizations are those that must be made before 
Drill starts. Setting the memory limit is a typical example. Here, drill-env.sh 
lets you set a value per-node by using a Linux command to set the host name:
    
        export DRILL_HOST_NAME=`hostname`
    
    The idea is that someone who needs your feature would add a line to 
drill-env.sh to set the proper "public" name of the host using a command 
appropriate to their setup. (Maybe using hostname, or, on EC2, using the 
appropriate Amazon-provided command.)
    
    The environment variable would pass the info into Drill where it would 
override the default.
    
    Because the host name must be different per node, and Drill is supposed to 
be distributed, it probably makes a bit less sense to set the host name in the 
Drill config file. (Would be cool if the config file supported scripts - but, 
alas, it does not.)
    
    Similar examples are the DRILL_CONFIG_DIR, DRILL_LOG_DIR, 
DRILL_JAVA_LIB_PATH, etc.


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to