Skip to content

Commit

Permalink
[#5846][FOLLOWUP] dev(build): Change the environment variable from `E…
Browse files Browse the repository at this point in the history
…NABLE_JDBC_AUTHORIZATION` to `ENABLE_SQL_BASE_AUTHORIATION` (#5888)

### What changes were proposed in this pull request?

Change the environment variable from `ENABLE_JDBC_AUTHORIZATION` to
`ENABLE_SQL_BASE_AUTHORIATION`

### Why are the changes needed?

This is a follow up pull reqeust

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

No need.
  • Loading branch information
jerqi authored Dec 17, 2024
1 parent 94c6a72 commit cb66b9b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dev/docker/hive/start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,9 @@ cp -f ${HIVE_TMP_CONF_DIR}/* ${HIVE_CONF_DIR}
sed -i "s/__REPLACE__HOST_NAME/$(hostname)/g" ${HADOOP_CONF_DIR}/core-site.xml
sed -i "s/__REPLACE__HOST_NAME/$(hostname)/g" ${HADOOP_CONF_DIR}/hdfs-site.xml

if [[ -n "${ENABLE_JDBC_AUTHORIZATION}" ]]; then
if [[ -n "${ENABLE_SQL_BASE_AUTHORIZATION}" ]]; then
if [[ -n "${RANGER_HIVE_REPOSITORY_NAME}" && -n "${RANGER_SERVER_URL}" ]]; then
echo "You can't set ENABLE_JDBC_AUTHORIZATION and RANGER_HIVE_REPOSITORY_NAME at the same time."
echo "You can't set ENABLE_SQL_BASE_AUTHORIZATION and RANGER_HIVE_REPOSITORY_NAME at the same time."
exit -1
fi
cp -f ${HIVE_CONF_DIR}/hive-site-for-sql-base-auth.xml ${HIVE_CONF_DIR}/hive-site.xml
Expand Down

0 comments on commit cb66b9b

Please sign in to comment.