Skip to content
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

[KYUUBI #6411] Grpc common support #6412

Closed
wants to merge 37 commits into from
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
dd39efd
fix-#4057 info:
davidyuan1223 Jan 13, 2023
86e4e1c
fix-#4057 info: modify the shellcheck errors file in ./bin
davidyuan1223 Jan 13, 2023
cb11935
Merge remote-tracking branch 'origin/fix-#4057' into fix-#4057
davidyuan1223 Jan 13, 2023
55a0a43
Merge pull request #10 from xiaoyuandajian/fix-#4057
davidyuan1223 Jan 13, 2023
c48ad38
remove the used blank lines
davidyuan1223 Mar 5, 2023
16237c2
Merge branch 'apache:master' into master
davidyuan1223 Apr 2, 2023
bfa6cbf
Merge branch 'apache:master' into master
davidyuan1223 Sep 22, 2023
e244029
Merge branch 'apache:master' into master
davidyuan1223 Sep 26, 2023
b616044
fix_4186
davidyuan1223 Sep 26, 2023
360d183
Merge branch 'master' into fix_4186
davidyuan1223 Sep 26, 2023
c83836b
Merge pull request #11 from davidyuan1223/fix_4186
davidyuan1223 Sep 26, 2023
40e80d9
Revert "fix_4186"
davidyuan1223 Sep 27, 2023
0925a4b
Merge pull request #12 from davidyuan1223/revert-11-fix_4186
davidyuan1223 Sep 27, 2023
2beccb6
Merge branch 'apache:master' into master
davidyuan1223 Sep 28, 2023
c8eb9a2
Merge branch 'apache:master' into master
davidyuan1223 Oct 10, 2023
56b91a3
fix_4186
davidyuan1223 Oct 10, 2023
57ec746
Merge pull request #13 from davidyuan1223/fix
davidyuan1223 Oct 10, 2023
72e7aea
Merge branch 'apache:master' into master
davidyuan1223 Oct 20, 2023
bcb0cf3
Merge remote-tracking branch 'origin/master'
davidyuan1223 Oct 26, 2023
8b51840
add common method to get session level config
davidyuan1223 Oct 26, 2023
6376466
Merge branch 'apache:master' into master
davidyuan1223 Dec 23, 2023
46001d4
Merge branch 'apache:master' into master
davidyuan1223 Jan 26, 2024
3d6c53b
add new module common-grpc
davidyuan1223 May 23, 2024
e16fd7f
fix_4186
davidyuan1223 Oct 10, 2023
a3a4ebd
add common method to get session level config
davidyuan1223 Oct 26, 2023
631ac9a
add new module common-grpc
davidyuan1223 May 23, 2024
0a52c2a
fork master new version
davidyuan1223 May 23, 2024
d92fdb6
Merge remote-tracking branch 'origin/grpc-common-support' into grpc-c…
davidyuan1223 May 23, 2024
4c5a4ae
fix format error
davidyuan1223 May 23, 2024
3e302aa
fix format error
davidyuan1223 May 23, 2024
c6af8ed
update settings.md
davidyuan1223 May 23, 2024
a1e0447
update
davidyuan1223 May 28, 2024
5bf1814
update
davidyuan1223 Jun 4, 2024
e031c8c
update
davidyuan1223 Jun 4, 2024
55cbae3
update
davidyuan1223 Jul 10, 2024
0c17121
update
davidyuan1223 Jul 15, 2024
2b959f9
add new module kyuubi-grpc
davidyuan1223 Jul 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix-#4057 info: modify the shellcheck errors file in ./bin
1. "$@" is a array, we want use string to compare. so update "$@" => "$*"
2. `tty` mean execute the command, we can use $(tty) replace it
3. param $# is a number, compare number should use -gt/-lt,not >/<
4. not sure the /bin/kyuubi line 63 'exit -1' need modify? so the directory bin only have a shellcheck note in /bin/kyuubi
  • Loading branch information
davidyuan1223 committed Jan 13, 2023
commit 86e4e1ce0baccc4ac56c57c6cc580badc55a1203
4 changes: 2 additions & 2 deletions bin/docker-image-tool.sh
Original file line number Diff line number Diff line change
Expand Up @@ -227,8 +227,8 @@ Examples:
EOF
}

# shellcheck disable=SC2199
if [[ "$@" = *--help ]] || [[ "$@" = *-h ]]; then

if [[ "$*" = *--help ]] || [[ "$*" = *-h ]]; then
usage
exit 0
fi
Expand Down
4 changes: 2 additions & 2 deletions bin/kyuubi
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ function usage() {
echo " -h | --help - Show this help message"
}

# shellcheck disable=SC2199
if [[ "$@" = *--help ]] || [[ "$@" = *-h ]]; then

if [[ "$*" = *--help ]] || [[ "$*" = *-h ]]; then
usage
exit 0
fi
Expand Down
8 changes: 4 additions & 4 deletions bin/kyuubi-logo
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,15 @@

# Bugzilla 37848: When no TTY is available, don't output to console
have_tty=0
# shellcheck disable=SC2006
if [[ "`tty`" != "not a tty" ]]; then

if [[ "$(tty)" != "not a tty" ]]; then
have_tty=1
fi

# Bugzilla 37848: When no TTY is available, don't output to console
have_tty=0
# shellcheck disable=SC2006
if [[ "`tty`" != "not a tty" ]]; then

if [[ "$(tty)" != "not a tty" ]]; then
have_tty=1
fi

Expand Down
4 changes: 2 additions & 2 deletions bin/stop-application.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
# limitations under the License.
#

# shellcheck disable=SC2071
if [[ $# < 1 ]] ; then

if [[ $# -lt 1 ]] ; then
echo "USAGE: $0 <application_id>"
exit 1
fi
Expand Down