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 <[email protected]>
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 <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 .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

Reply via email to