[KYUUBI #3435] Command should not execute when plan only mode is set to PHYSICAL or EXECUTION

[lsm1]

Why are the changes needed?

close #3435

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 locally before make a pull request

[wForget]

This change doesn't seem to be compatible with Spark 3.1, can you make it compatible?

[codecov-commenter]

Codecov Report

Merging #3439 (3447cbf) into master (b418f89) will increase coverage by 0.00%. The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #3439   +/-   ##
=========================================
  Coverage     51.92%   51.92%           
  Complexity       13       13           
=========================================
  Files           508      522   +14     
  Lines         29029    29049   +20     
  Branches       3985     3887   -98     
=========================================
+ Hits          15072    15084   +12     
- Misses        12532    12590   +58     
+ Partials       1425     1375   -50     

Impacted Files	Coverage Δ
...ubi/engine/spark/operation/PlanOnlyStatement.scala	39.78% <0.00%> (-3.24%)	:arrow_down:
...uubi/plugin/spark/authz/ranger/AccessRequest.scala	71.42% <0.00%> (-22.33%)	:arrow_down:
...mon/src/main/scala/org/apache/kyuubi/Logging.scala	43.42% <0.00%> (-9.22%)	:arrow_down:
.../main/scala/org/apache/kyuubi/util/SignUtils.scala	87.87% <0.00%> (-8.68%)	:arrow_down:
.../kyuubi/operation/KyuubiApplicationOperation.scala	40.90% <0.00%> (-6.46%)	:arrow_down:
.../kyuubi/server/mysql/constant/MySQLErrorCode.scala	13.84% <0.00%> (-6.16%)	:arrow_down:
...pache/kyuubi/engine/YarnApplicationOperation.scala	58.73% <0.00%> (-4.24%)	:arrow_down:
...apache/kyuubi/session/KyuubiBatchSessionImpl.scala	86.58% <0.00%> (-4.09%)	:arrow_down:
...ache/kyuubi/server/mysql/MySQLCommandHandler.scala	77.77% <0.00%> (-4.05%)	:arrow_down:
...uubi/plugin/spark/authz/util/SemanticVersion.scala	50.00% <0.00%> (-3.58%)	:arrow_down:
... and 68 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

[yaooqinn]

shall we remove this two? they are dangerous and not very useful

[lsm1]

SPARK-35378 add assertCommandExecuted since spark 3.2.0,spark 3.1 is normal. we can fix this case except execution mode in json style, can we just remove execution mode in json style?

[yaooqinn]

ok

[yaooqinn]

this introduces unstable private class from spark, not a good solution to me