Skip to content

Commit

Permalink
[KYUUBI #4191] [Improvement] Skip fetching Maven dependency snapshot …
Browse files Browse the repository at this point in the history
…update when building `dependencyList`

### _Why are the changes needed?_

- Skip Maven dependency snapshot update fetching to building `dependencyList` faster without waiting for fetching daily snapshots checking (only snapshots of submodules used).

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [x] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #4191 from bowenliang123/dependency-resolve.

Closes #4191

0bf3681 [liangbowen] apply `--no-snapshot-updates` option in build_classpath

Authored-by: liangbowen <liangbowen@gf.com.cn>
Signed-off-by: liangbowen <liangbowen@gf.com.cn>
  • Loading branch information
bowenliang123 committed Jan 30, 2023
1 parent b6a1a37 commit 7c036fb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build/dependency.sh
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ DEP="${PWD}/dev/dependencyList"


function build_classpath() {
$MVN dependency:build-classpath -pl kyuubi-ctl,kyuubi-server,kyuubi-assembly |\
$MVN dependency:build-classpath --no-snapshot-updates -pl kyuubi-ctl,kyuubi-server,kyuubi-assembly |\
grep -v "INFO\|WARN" | \
tail -1 | \
tr ":" "\n" | \
Expand Down

0 comments on commit 7c036fb

Please sign in to comment.