[
https://issues.apache.org/jira/browse/KYLIN-5233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17598700#comment-17598700
]
ASF GitHub Bot commented on KYLIN-5233:
---------------------------------------
Mukvin commented on code in PR #1961:
URL: https://github.com/apache/kylin/pull/1961#discussion_r960151400
##########
build/release/download-spark.sh:
##########
@@ -54,7 +54,7 @@ rm -rf build/spark/data
rm -rf build/spark/R
rm -rf build/spark/hive_1_2_2
-cp -rf build/hadoop3 build/spark/
+#cp -rf build/hadoop3 build/spark/
Review Comment:
here will add an external compatible jar for CDH 6.1, it can be removed.
##########
build/release/package.sh:
##########
@@ -20,58 +20,63 @@
dir=$(dirname ${0})
cd ${dir}/../..
-source build/apache_release/functions.sh
+source build/release/functions.sh
-echo "Packing for KE..."
+echo "Packing for Apache Kylin ..."
# Make share commands exist in environment
echo "BUILD STAGE 1 - Checking environment..."
checkCommandExists mvn
checkCommandExists git
checkCommandExists npm
+# Fetch ${release_version} from pom.xml
exportProjectVersions
-kap_commit_sha1=`git rev-parse HEAD`
-echo "${kap_commit_sha1}@KAP" > build/commit_SHA1
+kylin_commit_sha1=`git rev-parse HEAD`
+if [[ "${current_branch}" = "" ]]; then
+ current_branch=$(git branch | sed -n -e 's/^\* \(.*\)/\1/p')
+fi
+echo "${kylin_commit_sha1}@${current_branch}" > build/commit_SHA1
if [ -z "$BUILD_SYSTEM" ]; then
BUILD_SYSTEM="MANUAL"
fi
echo "Build with ${BUILD_SYSTEM} at" `date "+%Y-%m-%d %H:%M:%S"` >>
build/commit_SHA1
-KYLIN_VERSION_NAME="Kylin 5 ${release_version}"
+KYLIN_VERSION_NAME="Apache Kylin ${release_version}"
echo "${KYLIN_VERSION_NAME}" > build/VERSION
echo "VERSION file content:" ${KYLIN_VERSION_NAME}
echo "BUILD STAGE 2 - Build binaries..."
-sh build/apache_release/build.sh $@ || { exit 1; }
+sh build/release/build.sh $@ || { exit 1; }
if [[ "${WITH_SPARK}" = "1" ]]; then
echo "BUILD STAGE 3 - Prepare spark..."
- sh -x build/apache_release/download-spark.sh || { exit 1; }
+ sh build/release/download-spark.sh || { exit 1; }
Review Comment:
it is recommended to use with "-x"
> Add doc for How to debug in IDE
> -------------------------------
>
> Key: KYLIN-5233
> URL: https://issues.apache.org/jira/browse/KYLIN-5233
> Project: Kylin
> Issue Type: Sub-task
> Reporter: Xiaoxiang Yu
> Assignee: Xiaoxiang Yu
> Priority: Major
> Fix For: 5.0-alpha
>
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)