This is an automated email from the ASF dual-hosted git repository.

bowenliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new 10adbdc18 [KYUUBI #4315] [INFRA] Check modules available before build 
step in dependency workflow
10adbdc18 is described below

commit 10adbdc1855b8059fe2e7e72cf814aad2af63b20
Author: liangbowen <[email protected]>
AuthorDate: Mon Feb 13 20:11:20 2023 +0800

    [KYUUBI #4315] [INFRA] Check modules available before build step in 
dependency 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
    
    7c20ea24 [liangbowen] nit
    65397868 [liangbowen] add validate to bring back enforcer check
    1c035136 [liangbowen] add validate to bring back enforcer check
    d4e703de [liangbowen] update dep.yml
    8f8d0457 [liangbowen] update dep.yml
    e7adfacf [liangbowen] update
    c88eff98 [liangbowen] check modules available before build
    
    Authored-by: liangbowen <[email protected]>
    Signed-off-by: liangbowen <[email protected]>
---
 .github/workflows/dep.yml | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/.github/workflows/dep.yml b/.github/workflows/dep.yml
index dc94e4ea7..ebda6b47e 100644
--- a/.github/workflows/dep.yml
+++ b/.github/workflows/dep.yml
@@ -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

Reply via email to