kyuubi icon indicating copy to clipboard operation
kyuubi copied to clipboard

[KYUUBI #6353] Catch exception for closing flink internal session

Open wForget opened this issue 9 months ago โ€ข 1 comments

:mag: Description

Issue References ๐Ÿ”—

This pull request fixes #6353

Describe Your Solution ๐Ÿ”ง

Catch exception for closing flink internal session.

Types of changes :bookmark:

  • [X] Bugfix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan ๐Ÿงช

Behavior Without This Pull Request :coffin:

Behavior With This Pull Request :tada:

Related Unit Tests


Checklist ๐Ÿ“

Be nice. Be informative.

wForget avatar Apr 30 '24 09:04 wForget

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.45%. Comparing base (c73ae71) to head (32fc9af).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #6354   +/-   ##
=========================================
  Coverage     58.45%   58.45%           
  Complexity       24       24           
=========================================
  Files           653      653           
  Lines         39880    39880           
  Branches       5481     5481           
=========================================
+ Hits          23311    23312    +1     
- Misses        14071    14074    +3     
+ Partials       2498     2494    -4     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Apr 30 '24 11:04 codecov-commenter

Thanks, merged to master/1.9

pan3793 avatar Jun 06 '24 03:06 pan3793