Skip to content

Commit

Permalink
[KYUUBI #4315] [INFRA] Check modules available before build step in d…
Browse files Browse the repository at this point in the history
…ependency workflow

### _Why are the changes needed?_

- Check whether modules are available by running `mvnd dependency:resolve`. if true, skip the `build` step of `mvnd install`.
- Reducing the time cost of dependency workflow from  ~ 10 min to ~4 min ([see log](https://github.com/apache/kyuubi/actions/runs/4162071562/jobs/7200786535)).

### _How was this patch tested?_
- [x] Pass CI workflows

Closes #4315 from bowenliang123/dep-check-first.

Closes #4315

7c20ea2 [liangbowen] nit
6539786 [liangbowen] add validate to bring back enforcer check
1c03513 [liangbowen] add validate to bring back enforcer check
d4e703d [liangbowen] update dep.yml
8f8d045 [liangbowen] update dep.yml
e7adfac [liangbowen] update
c88eff9 [liangbowen] check modules available before build

Authored-by: liangbowen <liangbowen@gf.com.cn>
Signed-off-by: liangbowen <liangbowen@gf.com.cn>
  • Loading branch information
bowenliang123 committed Feb 13, 2023
1 parent 161dd3f commit 10adbdc
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions .github/workflows/dep.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,18 @@ jobs:
check-latest: false
- name: Setup Mvnd
uses: ./.github/actions/setup-mvnd
- name: Check kyuubi modules available
id: modules-check
run: >-
build/mvnd dependency:resolve validate
-DincludeGroupIds="org.apache.kyuubi" -DincludeScope="compile"
-Pfast -Denforcer.skip=false
-pl kyuubi-ctl,kyuubi-server,kyuubi-assembly -am
continue-on-error: true
- name: build
env:
MAVEN_OPTS: -Dorg.slf4j.simpleLogger.defaultLogLevel=error
if: steps.modules-check.conclusion == 'success' && steps.modules-check.outcome == 'failure'
run: >-
build/mvnd clean install
-Pflink-provided,spark-provided,hive-provided
Expand Down

0 comments on commit 10adbdc

Please sign in to comment.