Leemoonsoo commented on pull request #4070:
URL: https://github.com/apache/zeppelin/pull/4070#issuecomment-790873827


   > I think this is the first step in the right direction. I think we should 
replace Z_HOME completely with HOME.
   
   How about rename `Z_HOME` to `ZEPPELIN_HOME`, and keep the `HOME` env as 
well?
   The reason is, `ZEPPELIN_HOME` is already being used in many scripts like 
[bin/zeppelin.sh](https://github.com/apache/zeppelin/blob/master/bin/zeppelin.sh).
   
   And `HOME` is still required to make some external command work (`yarn` cmd 
for installing Helium package),
   but doesn't necessarily have to be the same as `ZEPPELIN_HOME`.
   
   What do you think?
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to