unknowntpo commented on code in PR #6848:
URL: https://github.com/apache/gravitino/pull/6848#discussion_r2032464609


##########
dev/docker/ranger/ranger-dependency.sh:
##########
@@ -36,7 +36,36 @@ if [[ ! -d "${ranger_dir}/packages" ]]; then
 fi
 
 if [ ! -f "${ranger_dir}/packages/${RANGER_PACKAGE_NAME}" ]; then
-  curl -L -s -o "${ranger_dir}/packages/${RANGER_PACKAGE_NAME}" 
${RANGER_DOWNLOAD_URL}
+  # Package not exist, we need to build them from source
+       if [ ! -d "${ranger_dir}/packages/apache-ranger" ]; then
+               git clone https://github.com/apache/ranger --branch master 
--single-branch ${ranger_dir}/packages/apache-ranger
+               # set the commit to RANGER-5146: 500 API Error When Deleting 
TagDef with a Linked Tag
+               # 
https://github.com/apache/ranger/commit/ff36aabe36169b94862c51a5b403f59c9d728b94
+               git reset --hard ff36aabe36169b94862c51a5b403f59c9d728b94
+       fi
+
+       cp ${ranger_dir}/.env 
${ranger_dir}/packages/apache-ranger/dev-support/ranger-docker
+
+       cd ${ranger_dir}/packages/apache-ranger/dev-support/ranger-docker
+
+  # Prevent builder to pull remote image
+  # https://github.com/moby/buildkit/issues/2343#issuecomment-1311890308
+  docker builder prune -f
+  docker builder use default

Review Comment:
   I think these two lines are dealing with the same thing, so I don't add new 
line between them.
   



-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@gravitino.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to