stoty commented on a change in pull request #959:
URL: https://github.com/apache/phoenix/pull/959#discussion_r522812170
##########
File path: dev/create-release/release-util.sh
##########
@@ -636,12 +637,22 @@ function kick_gpg_agent {
# Do maven command to set version into local pom
function maven_set_version { #input: <version_to_set>
local this_version="$1"
- echo "${MVN[@]}" versions:set -DnewVersion="$this_version"
- "${MVN[@]}" versions:set -DnewVersion="$this_version" | grep -v "no value" #
silence logs
- #Hacking around nonstandard maven submodules
+ # Omid needs an hbase profile for tagging (doesn't matter which one, the
shims need special handling anyway)
+ if [[ "$PROJECT" == "phoenix-omid" ]]; then
+ local variant="${OMID_VARIANTS[1]}"
+ fi
+ echo "${MVN[@]}" ${variant:+"$variant"} versions:set
-DnewVersion="$this_version"
+ "${MVN[@]}" ${variant:+"$variant"} versions:set -DnewVersion="$this_version"
| grep -v "no value" # silence logs
+ #Hacking around nonstandard maven submodules for phoenix only
for i in phoenix-hbase-compat-*; do
if [ -e "$i" ]; then
- "${MVN[@]}" -pl $i versions:set -DnewVersion="$this_version" | grep -v
"no value" # silence logs
+ "${MVN[@]}" ${variant:+"$variant"} -pl $i versions:set
-DnewVersion="$this_version" | grep -v "no value" # silence logs
+ fi
+ done
+ #Omid has an even more nonstandard maven structure, and needs more hacks
+ for i in hbase-shims/hbase-*; do
+ if [ -e "$i" ]; then
+ sed -i -e
"0,\#<version>.*</version>#{s##<version>${this_version}</version>#}" $i/pom.xml
Review comment:
Long term we should fix the build to build both set of artifacts in one
pass, like connectors and tephra does.
----------------------------------------------------------------
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]
