[SENTRY-430] Sentry Service does not use correct classpath when HIVE_HOME environment var is defined

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.4.0
• Fix Version/s: 1.5.0
• Assignee: Prasad Suresh Mujumdar
• Reporter: Lenni Kuff
• Created: 11/Sep/14 20:08
• Updated: 18/Mar/15 05:21
• Resolved: 19/Sep/14 05:52

Description

When HIVE_HOME is set as an environment variable, the Sentry Service will not run with the correct classpath when invoked using "bin/sentry".

The code below is from "bin/sentry". Note how if HIVE_HOME is defined nothing from lib/server/* is added to the classpath.

As a side note, it seems strange that there even is a reference to Hive in this context. It seems that the sentry service is being executed via Hive when HIVE_HOME is defined. It seems like we should not do this.

if [[ -z "$HIVE_HOME" ]]
then
  for f in ${SENTRY_HOME}/lib/server/*.jar; do
    HADOOP_CLASSPATH=${HADOOP_CLASSPATH}:${f}
  done
  exec $HADOOP jar ${SENTRY_HOME}/lib/${_CMD_JAR} org.apache.sentry.SentryMain ${args[@]}
else
  _HIVE_CMD=${HIVE_HOME}/bin/hive
  ${_HIVE_CMD} --service jar ${SENTRY_HOME}/lib/${_CMD_JAR} org.apache.sentry.SentryMain ${args[@]}
fi

Issue Links

• links to: Review #25799