kyuubi icon indicating copy to clipboard operation
kyuubi copied to clipboard

Initial implement Arrow Flight SQL FE

Open pan3793 opened this issue 3 years ago • 3 comments

Why are the changes needed?

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

  • [ ] Run test locally before make a pull request

pan3793 avatar Aug 28 '22 23:08 pan3793

Codecov Report

Merging #3364 (59f762b) into master (f9cc709) will increase coverage by 0.01%. The diff coverage is 55.94%.

@@             Coverage Diff              @@
##             master    #3364      +/-   ##
============================================
+ Coverage     51.47%   51.49%   +0.01%     
  Complexity       13       13              
============================================
  Files           475      478       +3     
  Lines         26490    26631     +141     
  Branches       3698     3700       +2     
============================================
+ Hits          13637    13713      +76     
- Misses        11525    11585      +60     
- Partials       1328     1333       +5     
Impacted Files Coverage Δ
...kyuubi/server/filght/KyuubiFlightSQLProducer.scala 11.94% <11.94%> (ø)
.../scala/org/apache/kyuubi/server/KyuubiServer.scala 54.41% <66.66%> (+0.56%) :arrow_up:
...kyuubi/server/KyuubiFlightSQLFrontendService.scala 95.23% <95.23%> (ø)
...in/scala/org/apache/kyuubi/config/KyuubiConf.scala 96.95% <96.42%> (-0.02%) :arrow_down:
...a/org/apache/kyuubi/server/filght/ArrowUtils.scala 100.00% <100.00%> (ø)
...ain/scala/org/apache/kyuubi/engine/EngineRef.scala 74.07% <0.00%> (-0.93%) :arrow_down:
...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala 89.27% <0.00%> (-0.70%) :arrow_down:
.../engine/spark/session/SparkSQLSessionManager.scala 79.74% <0.00%> (+1.26%) :arrow_up:

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

codecov-commenter avatar Aug 29 '22 01:08 codecov-commenter

Do we have a umbrella ticket for this?

Cc @lightning-L

turboFei avatar Aug 30 '22 04:08 turboFei

Do we have a umbrella ticket for this?

yes, https://github.com/apache/incubator-kyuubi/issues/3356, linked.

pan3793 avatar Aug 30 '22 04:08 pan3793

Thanks for the PR! This PR is being closed due to inactivity. This isn't a judgement on the merit of the PR in any way. If this is still an issue with the latest version of Kyuubi, please reopen it and ask a committer to remove the Stale tag!

Thank you for using Kyuubi!

github-actions[bot] avatar Dec 09 '22 00:12 github-actions[bot]