-
Notifications
You must be signed in to change notification settings - Fork 994
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHOENIX-6216 Add Omid and Tephra support to create-release script #959
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@@ -52,6 +52,11 @@ RUN wget -qO- "https://www.apache.org/dyn/mirrors/mirrors.cgi?action=download&fi | |||
tar xvz -C /opt | |||
ENV YETUS_HOME /opt/apache-yetus-${YETUS_VERSION} | |||
|
|||
# Install protobuf-2.5 protoc binary from maven central, as that seems to be the only binary available | |||
RUN mkdir /opt/protobuf-2.5.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not particular to this change, but I hope Omid dependency on protobuf 2.5.0 doesn't conflict with the protobuf 3.0+ in Hadoop/HBase/Phoenix runtime.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't make the HBase protobuf version mess any worse than it already is :)
We use unshaded 2.5 for the coprocessor api, just as in Phoenix.
#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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to find the alternative with options in the version maven plugin to update multi-level parent-child relationship but it seems we have no option other than this hack
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Long term we should fix the build to build both set of artifacts in one pass, like connectors and tephra does.
No description provided.