-
Notifications
You must be signed in to change notification settings - Fork 416
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
[#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 #6652
Changes from 11 commits
8a8577b
445bc7c
17f68d0
b032aa6
8c96734
32adfe7
b134f95
c644e05
4d1641b
f63b6ba
b05b8cd
fffeca0
3e8b552
53c0f97
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,72 @@ | ||
#!/usr/bin/env bash | ||
# Licensed to the Apache Software Foundation (ASF) under one | ||
# or more contributor license agreements. See the NOTICE file | ||
# distributed with this work for additional information | ||
# regarding copyright ownership. The ASF licenses this file | ||
# to you under the Apache License, Version 2.0 (the | ||
# "License"); you may not use this file except in compliance | ||
# with the License. You may obtain a copy of the License at | ||
# | ||
# http://www.apache.org/licenses/LICENSE-2.0 | ||
# | ||
# Unless required by applicable law or agreed to in writing, software | ||
# distributed under the License is distributed on an "AS IS" BASIS, | ||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
# See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
|
||
# | ||
# gh-dl-release! It works! | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what is gh-dl-release? |
||
# | ||
# This script downloads an asset from latest or specific Github release of a | ||
# private repo. Feel free to extract more of the variables into command line | ||
# parameters. | ||
# | ||
# PREREQUISITES | ||
# | ||
# curl, wget, jq | ||
# | ||
# USAGE | ||
# | ||
# Set all the variables inside the script, make sure you chmod +x it, then | ||
# to download specific version to my_app.tar.gz: | ||
# | ||
# gh-dl-release 2.1.1 my_app.tar.gz | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
I'm thinking something like this:
|
||
# | ||
# to download latest version: | ||
# | ||
# gh-dl-release latest latest.tar.gz | ||
# | ||
# If your version/tag doesn't match, the script will exit with error. | ||
set -ex | ||
|
||
TOKEN=${PRIVATE_ACCESS_TOKEN} | ||
REPO="datastrato/ranger" | ||
FILE=$2 #"ranger-2.4.0-admin.tar.gz" # the name of your release asset file, e.g. build.tar.gz | ||
VERSION=$1 # tag name or the word "latest" | ||
GITHUB="https://api.github.com" | ||
|
||
alias errcho='>&2 echo' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not really; it stands for There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If it is not used commonly and made by you, I would suggest you pick a better name. |
||
|
||
function gh_curl() { | ||
curl -H "Authorization: token $TOKEN" \ | ||
-H "Accept: application/vnd.github.v3.raw" \ | ||
$@ | ||
} | ||
|
||
if [ "$VERSION" = "latest" ]; then | ||
# Github should return the latest release first. | ||
parser=".[0].assets | map(select(.name == \"$FILE\"))[0].id" | ||
else | ||
parser=". | map(select(.tag_name == \"$VERSION\"))[0].assets | map(select(.name == \"$FILE\"))[0].id" | ||
fi; | ||
|
||
asset_id=`gh_curl -s $GITHUB/repos/$REPO/releases | jq "$parser"` | ||
if [ "$asset_id" = "null" ]; then | ||
errcho "ERROR: version not found $VERSION" | ||
exit 1 | ||
fi; | ||
|
||
echo $2 | ||
|
||
curl -sL --header "Authorization: token $TOKEN" --header 'Accept: application/octet-stream' https://$TOKEN:@api.github.com/repos/$REPO/releases/assets/$asset_id -o $3 |
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -185,6 +185,11 @@ You can use this kind of image to test the catalog of Apache Hive. | |||||||
|
||||||||
Changelog | ||||||||
|
||||||||
|
||||||||
- apache/gravitino-ci:hive-0.1.18 | ||||||||
- Support UTF-8 encoding for the `hive-site.xml` file and Hive Metastore. For more information, please see [PR](https://github.com/apache/gravitino/pull/6625) | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
- Change ranger-hive-plugin and ranger-hdfs-plugin download URL. | ||||||||
|
||||||||
- apache/gravitino-ci:hive-0.1.17 | ||||||||
- Add support for JDBC SQL standard authorization | ||||||||
- Add JDBC SQL standard authorization related configuration in the `hive-site-for-sql-base-auth.xml` and `hiveserver2-site-for-sql-base-auth.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.
@yuqi1129 Please clean invalid comments.
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.
done