You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2023/06/01 03:46:10 UTC

[kyuubi] branch master updated: [KYUUBI #4909] [AUTHZ] Enable authz plugin tests for Spark 3.4

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

chengpan 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 f97928217 [KYUUBI #4909] [AUTHZ] Enable authz plugin tests for Spark 3.4
f97928217 is described below

commit f979282179e06f0ffdd915b332611a99c467d1dc
Author: liangbowen <li...@gf.com.cn>
AuthorDate: Thu Jun 1 11:46:00 2023 +0800

    [KYUUBI #4909] [AUTHZ] Enable authz plugin tests for Spark 3.4
    
    ### _Why are the changes needed?_
    
    Authz plugin's tests now pass with Spark3.4.
    
    ` mvn clean install -pl :kyuubi-spark-authz_2.12 -Pspark-3.4 -am -DskipTests
     &&  mvn test -pl :kyuubi-spark-authz_2.12 -Pspark-3.4`
    
    ![image](https://github.com/apache/kyuubi/assets/1935105/3cf8b43a-6aca-427a-bfa3-c06caef0289a)
    
    ### _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 #4909 from bowenliang123/authz-enabletest-3.4.
    
    Closes #4909
    
    b0bdc106e [liangbowen] enable authz plugin tests with Spark 3.4
    
    Authored-by: liangbowen <li...@gf.com.cn>
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 .github/workflows/master.yml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/.github/workflows/master.yml b/.github/workflows/master.yml
index 99817e6ca..674c7d060 100644
--- a/.github/workflows/master.yml
+++ b/.github/workflows/master.yml
@@ -102,8 +102,6 @@ jobs:
         run: |
           TEST_MODULES="dev/kyuubi-codecov"
           if [[ "${{ matrix.spark }}" == "3.4" ]]; then
-            # FIXME: Spark 3.4 supports authz plugin
-            TEST_MODULES="$TEST_MODULES,!extensions/spark/kyuubi-spark-authz"
             # FIXME: Spark 3.4 supports lineage plugin
             TEST_MODULES="$TEST_MODULES,!extensions/spark/kyuubi-spark-lineage"
           fi