[KYUUBI #6084] JDBC driver supports configuration not to pull engine startup logs

[lsm1]

🔍 Description

Issue References 🔗

This pull request fixes #6084

Describe Your Solution 🔧

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Support --ignore-launch-engine option and not pull engine starup logs

Types of changes 🔖

• 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 ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests

---

Checklist 📝

• This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

[pan3793]

is it for security purposes? if so, maybe we should disable the log on the server side?

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 62.25%. Comparing base (2c9fb3d) to head (a9c08ad).
Report is 35 commits behind head on master.

Files	Patch %	Lines
.../org/apache/kyuubi/jdbc/hive/KyuubiConnection.java	33.33%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #6106      +/-   ##
============================================
+ Coverage     61.12%   62.25%   +1.13%     
- Complexity       23       24       +1     
============================================
  Files           623      625       +2     
  Lines         37200    38982    +1782     
  Branches       5040     5333     +293     
============================================
+ Hits          22738    24269    +1531     
- Misses        12001    12245     +244     
- Partials       2461     2468       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cxzl25]

is it for security purposes?

We just want to align hive's native beeline behavior and reduce some logs and RPC.

[pan3793]

beeline has a --verbose option, seems we do not respect it though

[zhouyifan279]

Can we skip the creation of engine-launch-log thread created byKyuubiConnection#showLaunchEngineLog if ignoreLaunchEngineLog is true ?