kyuubi
kyuubi copied to clipboard
[KYUUBI #3220] Make kyuubi.engine.ui.stop.enabled false in HistoryServer
Why are the changes needed?
close #3220
How was this patch tested?
-
[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
-
[X] Add screenshots for manual tests if appropriate

-
[ ] Run test locally before make a pull request
Codecov Report
Merging #3221 (ed3f596) into master (6aa898e) will increase coverage by
0.06%. The diff coverage is76.19%.
@@ Coverage Diff @@
## master #3221 +/- ##
============================================
+ Coverage 51.56% 51.62% +0.06%
Complexity 6 6
============================================
Files 459 464 +5
Lines 25556 25652 +96
Branches 3545 3551 +6
============================================
+ Hits 13177 13244 +67
- Misses 11121 11140 +19
- Partials 1258 1268 +10
| Impacted Files | Coverage Δ | |
|---|---|---|
| ...rg/apache/spark/ui/KyuubiHistoryServerPlugin.scala | 0.00% <0.00%> (ø) |
|
| ...rg/apache/kyuubi/events/EventHandlerRegister.scala | 60.00% <60.00%> (ø) |
|
| ...rg/apache/kyuubi/engine/spark/SparkSQLEngine.scala | 76.87% <100.00%> (-0.01%) |
:arrow_down: |
| ...ngine/spark/events/SparkEventHandlerRegister.scala | 100.00% <100.00%> (ø) |
|
| ...che/kyuubi/events/ServerEventHandlerRegister.scala | 100.00% <100.00%> (ø) |
|
| .../scala/org/apache/kyuubi/server/KyuubiServer.scala | 53.84% <100.00%> (-4.60%) |
:arrow_down: |
| ...ain/scala/org/apache/kyuubi/engine/EngineRef.scala | 74.07% <0.00%> (-2.12%) |
:arrow_down: |
| ...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala | 78.00% <0.00%> (-2.00%) |
:arrow_down: |
| ...pache/kyuubi/engine/YarnApplicationOperation.scala | 64.81% <0.00%> (-1.86%) |
:arrow_down: |
| ...mon/src/main/scala/org/apache/kyuubi/Logging.scala | 43.42% <0.00%> (-1.32%) |
:arrow_down: |
| ... and 10 more |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Thanks, merging to master