-
Notifications
You must be signed in to change notification settings - Fork 50
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
Fix ssl certificate error and broken Hive download link #199
Conversation
ebyhr
commented
May 23, 2024
•
edited
Loading
edited
c9e92b7
to
4aa8649
Compare
Hit another issue. Looking for the new download link:
edit: https://dlcdn.apache.org/hive/hive-3.1.3/apache-hive-3.1.3-bin.tar.gz seems the new link. I can't find version 3.1.2. |
@@ -36,11 +36,11 @@ RUN yum install -y \ | |||
ENV JAVA_HOME=/usr/lib/jvm/zulu-8 | |||
|
|||
ARG HADOOP_VERSION=3.1.2 | |||
ARG HIVE_VERSION=3.1.2 | |||
ARG HIVE_VERSION=3.1.3 |
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.
should we run some tests locally with this before merging?
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.
Yes, I ran the following command and all tests passed:
testing/bin/ptl suite run --config config-apache-hive3 --suite suite-hms-only
...
tests | ===============================================
tests | tempto-tests
tests | Total tests run: 81, Failures: 0, Skips: 0
tests | ===============================================
the ssl related fix LGTM |
Also, update the download link.